Hi Paul, > +2024-04-27 Paul Eggert <egg...@cs.ucla.edu> > + > + stddef: work around GCC 14 stddef.h bugs > + * lib/stddef.in.h: Do nothing if _@GUARD_PREFIX@_STDDEF_H is > + defined, as stddef.h has already been included. This works > + around GCC bug 114870. > + (_GCC_NULLPTR_T): Define if needed to work around GCC bug 114869. > + * m4/stddef_h.m4 (gl_STDDEF_H, gl_STDDEF_H_DEFAULTS): > + * modules/stddef (stddef.h): > + Detect the two bugs.
I would have preferred if you had committed each workaround in a separate patch, because - One of the two workarounds causes the major trouble on Cygwin, reported by Markus Mützel, therefore I have to revert it. - Also the fact that the documentation mentions one of the two workarounds but not the other adds to the confusion. Let me start by adding a reference from the doc to the GCC bug tracker. diff --git a/ChangeLog b/ChangeLog index 118eeaa050..8681066eee 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2024-05-01 Bruno Haible <br...@clisp.org> + + doc: Reference a gcc bug. + * doc/posix-headers/stddef.texi: Reference a gcc bug. + 2024-05-01 Bruno Haible <br...@clisp.org> sprintf-posix, snprintf-posix tests: Add comment. @@ -169,6 +174,7 @@ * m4/stddef_h.m4 (gl_STDDEF_H, gl_STDDEF_H_DEFAULTS): * modules/stddef (stddef.h): Detect the two bugs. + * doc/posix-headers/stddef.texi: Mention one of the two bugs. 2024-04-27 Bruno Haible <br...@clisp.org> diff --git a/doc/posix-headers/stddef.texi b/doc/posix-headers/stddef.texi index 00860bade0..4d0e171b8d 100644 --- a/doc/posix-headers/stddef.texi +++ b/doc/posix-headers/stddef.texi @@ -48,6 +48,7 @@ @item Some platforms define @code{nullptr_t} even when @code{<stddef.h>} is not included: +@c https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114869 GCC 14.0.1 20240411 (Red Hat 14.0.1-0). @item