Page MenuHomeFreeBSD
Differential D47574

cdefs: Allow _XOPEN_SOURCE >= 800
AcceptedPublic
Actions

Authored by imp on Thu, Nov 14, 6:59 PM.

Details

Reviewers
kevans
brooks
Summary

When the user asks for XOPEN_SOURCE >= 800, set __XSI_VISIBLE to 800 for
POSIX 2024.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 60567
Build 57451: arc lint + arc unit

Event Timeline

imp requested review of this revision.Thu, Nov 14, 6:59 PM
imp created this revision.
Harbormaster completed remote builds in B60562: Diff 146457.Thu, Nov 14, 6:59 PM
imp updated this revision to Diff 146462.Thu, Nov 14, 6:59 PM
imp added a child revision: D47575: cdesf: Support _POSIX_C_SOURCE 202405 for POSIX 2024.

forgotten fold

Harbormaster completed remote builds in B60567: Diff 146462.Thu, Nov 14, 7:00 PM
brooks accepted this revision.Thu, Nov 14, 7:20 PM
brooks added a subscriber: brooks.

I wish there was a comment justifying the - 0 pattern...

This revision is now accepted and ready to land.Thu, Nov 14, 7:20 PM
imp added a comment.Thu, Nov 14, 10:11 PM
In D47574#1085131, @brooks wrote:

I wish there was a comment justifying the - 0 pattern...

I'll add one in a future commit.

imp added a comment.Thu, Nov 14, 10:38 PM
In D47574#1085196, @imp wrote:
In D47574#1085131, @brooks wrote:

I wish there was a comment justifying the - 0 pattern...

I'll add one in a future commit.

At this point, it's likely a historic practice we can drop. Peter Wemm added it 22 years
ago:
Deal with "#define _XOPEN_SOURCE" in header files (eg: XFree86 4.2.0)
so I'll add a comment to that effect. I'd just drop it, but a code search says it's
still a common idium.

imp added a comment.Thu, Nov 14, 11:39 PM

https://reviews.freebsd.org/D47584 ads the comment that Brooks suggested.

Revision Contents
Changeset List

PathSize
sys/
sys/
6 lines

Diff 146462

View Options

sys/sys/cdefs.h

Loading...