Skip to content

Remove generated Inventor/system/inttypes header#5

Merged
tritao merged 1 commit into
FreeCAD:freecad-masterfrom
tritao:remove-inttypes-header
Jul 9, 2026
Merged

Remove generated Inventor/system/inttypes header#5
tritao merged 1 commit into
FreeCAD:freecad-masterfrom
tritao:remove-inttypes-header

Conversation

@tritao

@tritao tritao commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

This removes the generated include/Inventor/system/inttypes.h compatibility header and the associated configure/build plumbing.

Coin now exposes the standard fixed-width and size-related types from Inventor/C/basic.h via <stdint.h> and <stddef.h>, which already sits under SbBasic.h and the public C API headers. The few implementation-only cases that need more than that keep direct standard includes locally:

  • <inttypes.h> for SCN* macros
  • <sys/types.h> where off_t is used

Why

Inventor/system/inttypes.h is legacy compatibility machinery that we do not need anymore. We can just assume the standard headers exist and stop carrying a generated wrapper plus all the related configure/install logic.

This also keeps the public header cleanup centralized instead of spreading direct <stdint.h> includes across the API surface.

Included cleanup

This change also removes the obsolete integer-type detection/configuration pieces that only existed to support the generated header, including:

  • the inttypes.h template files
  • the related CMake and Automake install/build rules
  • the old HAVE_* / COIN_* integer typedef plumbing in config templates
  • stale comments and sanity checks that depended on that machinery

@tritao
tritao force-pushed the remove-inttypes-header branch from 1de5665 to d383623 Compare July 9, 2026 14:00
@tritao
tritao marked this pull request as ready for review July 9, 2026 17:15
@tritao
tritao merged commit a9e0414 into FreeCAD:freecad-master Jul 9, 2026
5 checks passed
@Lgt2x

Lgt2x commented Jul 9, 2026

Copy link
Copy Markdown
Member

bumped somewhere in FC?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants