[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] stddef: work around GCC 14 stddef.h bugs
From: |
Bruno Haible |
Subject: |
Re: [PATCH 1/2] stddef: work around GCC 14 stddef.h bugs |
Date: |
Thu, 02 May 2024 00:17:58 +0200 |
Hi Paul,
> +2024-04-27 Paul Eggert <eggert@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 <bruno@clisp.org>
+
+ doc: Reference a gcc bug.
+ * doc/posix-headers/stddef.texi: Reference a gcc bug.
+
2024-05-01 Bruno Haible <bruno@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 <bruno@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
- Re: [PATCH 1/2] stddef: work around GCC 14 stddef.h bugs,
Bruno Haible <=