Paths

Table of Contentst

Differential D54043

headers: Remove redundant checks for __BSD_VISIBLE
Needs ReviewPublic
Actions

Authored by imp on Dec 3 2025, 4:46 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 6, 4:59 AM

	Unknown Object (File)
	Sat, Sep 5, 5:27 AM

	Unknown Object (File)
	Wed, Aug 26, 2:49 AM

	Unknown Object (File)
	Mon, Aug 10, 4:45 PM

	Unknown Object (File)
	Mon, Aug 10, 10:21 AM

	Unknown Object (File)
	Aug 9 2026, 5:47 AM

	Unknown Object (File)
	Aug 8 2026, 9:47 AM

	Unknown Object (File)
	Aug 2 2026, 3:38 AM

View All 33 Files

Subscribers

Details

Reviewers

andrew
manu
jrtc27
fuz

Summary

We assume that __BSD_VISIBLE is always defined when different subsetting
functions are in place. No need to test for it in these cases. The only
time you should test for it is when POSIX, ISOC or XPG has removed
functionality in newer releases and we omit defining those functions,
defines, etc in that case.

Sponsored by: Netflix

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 69007
Build 65890: arc lint + arc unit

Event Timeline

imp created this revision.Dec 3 2025, 4:46 AM

Herald added a reviewer: andrew. · View Herald TranscriptDec 3 2025, 4:46 AM

Herald added a reviewer: andrew. · View Herald Transcript

Herald added a reviewer: manu. · View Herald Transcript

Herald added subscribers: riscv, jhibbits, emaste. · View Herald Transcript

imp requested review of this revision.Dec 3 2025, 4:46 AM

Harbormaster completed remote builds in B69007: Diff 167462.Dec 3 2025, 4:46 AM

imp added reviewers: jrtc27, fuz.Dec 3 2025, 4:47 AM

fuz added inline comments.Dec 3 2025, 12:56 PM

sys/arm64/include/setjmp.h
68	Shouldn't `__XSI_VISIBLE` imply `__POSIX_VISIBLE`?

Revision Contents
Changeset List

Path

Size

include/

2 lines

2 lines

4 lines

lib/

msun/

src/

16 lines

sys/

arm/

include/

2 lines

arm64/

include/

2 lines

powerpc/

include/

2 lines

riscv/

include/

2 lines

sys/

9 lines

2 lines

x86/

include/

2 lines

Diff 167462

include/stdckdint.h

Loading...

include/string.h

Loading...

include/strings.h

Loading...

lib/msun/src/math.h

Loading...

sys/arm/include/setjmp.h

Loading...

sys/arm64/include/setjmp.h

Loading...

sys/powerpc/include/setjmp.h

Loading...

sys/riscv/include/setjmp.h

Loading...

sys/sys/_endian.h

Loading...

sys/sys/signal.h

Loading...

sys/x86/include/setjmp.h

Loading...