PR/57806 CVS commit: [netbsd-10] src/sys/sys

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR standards/57806; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57806 CVS commit: [netbsd-10] src/sys/sys
Date: Fri, 11 Oct 2024 19:07:20 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Fri Oct 11 19:07:20 UTC 2024
 
 Modified Files:
 	src/sys/sys [netbsd-10]: endian.h
 
 Log Message:
 Pull up following revision(s) (requested by riastradh in ticket #952):
 
 	sys/sys/endian.h: revision 1.32
 	sys/sys/endian.h: revision 1.33
 	sys/sys/endian.h: revision 1.34
 	sys/sys/endian.h: revision 1.35
 
 break the cycle: this does not need <sys/types.h> it only needs <sys/stdint.h>
 
 arpa/inet.h: Define htonl &c. under _POSIX_C_SOURCE >= 200112L too.
 These do not require _XOPEN_SOURCE as of POSIX 2001:
 https://pubs.opengroup.org/onlinepubs/009696799/functions/htonl.html
 
 XXX Since this change is actually sys/endian.h, in principle it might
 bleed into header files than just arpa/inet.h.  But sys/endian.h is
 not used by anything that looks especially risky to me.
 
 PR standards/57806: #include <arpa/inet.h> fails to define
 htonl/htons/ntohl/ntohs under _POSIX_C_SOURCE=200809L
 
 sys/endian.h: Hide le32enc/be32enc/... under _NETBSD_SOURCE.
 These are non-standard extensions, so they should not be exposed by,
 e.g., _XOPEN_SOURCE=700.
 
 PR standards/57807: #include <arpa/inet.h> spuriously defines
 le32enc/be32enc/... under _XOPEN_SOURCE=700
 
 sys/endian.h: sync comment with code
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.31 -r1.31.4.1 src/sys/sys/endian.h
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.