Page MenuHomeFreeBSD

libsysdecode: Include required headers in sysdecode.h
ClosedPublic

Authored by markj on Apr 13 2022, 2:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 23, 3:54 AM
Unknown Object (File)
Tue, Oct 22, 10:41 AM
Unknown Object (File)
Sun, Oct 20, 7:14 AM
Unknown Object (File)
Sat, Oct 19, 3:30 PM
Unknown Object (File)
Oct 18 2024, 1:17 AM
Unknown Object (File)
Oct 7 2024, 11:59 PM
Unknown Object (File)
Oct 7 2024, 9:18 AM
Unknown Object (File)
Oct 2 2024, 10:11 AM

Details

Summary

It seems a bit silly to force all consumers to include several system
headers any time they include sysdecode.h. There has been a trend in
FreeBSD to make headers more self-contained, so let's apply that here.
No functional change intended.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Apr 13 2022, 2:34 PM
This revision is now accepted and ready to land.Apr 13 2022, 2:37 PM

Manual page changes appear consistent with addition to sysdecode.h. (Naive question: does always including stdio.h cause any noticeable bloat?)

(Naive question: does always including stdio.h cause any noticeable bloat?)

I'm not sure what you mean by bloat. The only negative effect of this is that a consumers which already includes stdio.h will force the C preprocessor to read the file again. This has no impact on the compiled code but forces the C preprocessor to read the file again.