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




Reply via email to