HomeFreeBSD

systm.h: Include cdefs.h first

Description

systm.h: Include cdefs.h first

Ever since r143063, machine/atomic.h requires cdefs.h. So, include it
first. Weak support: style(9) tells us to include cdefs.h first.

Argument against: since code that includes systm.h still compiles,
compilation units that include systm.h must already include cdefs.h. So, an
argument could be made that the cdefs.h include could just be removed
entirely. That is maybe a bigger change and not one I am interested in
bikeshedding.

Universe compiles.

Sponsored by: Dell EMC Isilon

Details

Provenance
cemAuthored on Nov 10 2017, 2:00 AM
Parents
rGa264cb726bfb: Some fixups to the CFI directives for PLT stub entry points.
Branches
Unknown
Tags
Unknown

Event Timeline