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)
Thu, Nov 27, 4:55 PM
Unknown Object (File)
Thu, Nov 27, 4:29 PM
Unknown Object (File)
Thu, Nov 27, 11:53 AM
Unknown Object (File)
Sun, Nov 23, 10:58 PM
Unknown Object (File)
Tue, Nov 18, 3:31 PM
Unknown Object (File)
Mon, Nov 17, 7:16 PM
Unknown Object (File)
Sat, Nov 15, 1:14 AM
Unknown Object (File)
Fri, Nov 14, 10:27 PM

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
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45145
Build 42033: arc lint + arc unit

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.