On Wed, 5 Nov 2025 at 11:29, Xavier Bonaventura <[email protected]> wrote: > > SIG_IGN also needs to be defined according to the C++ standard. > This was missing in the test. > > libstdc++-v3/testsuite/ChangeLog: > > * 18_support/headers/csignal/macros.cc: New test. > > Signed-off-by: Xavier Bonaventura <[email protected]> > --- > libstdc++-v3/testsuite/18_support/headers/csignal/macros.cc | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/libstdc++-v3/testsuite/18_support/headers/csignal/macros.cc > b/libstdc++-v3/testsuite/18_support/headers/csignal/macros.cc > index 6d6c8217b36..1076cbca24d 100644 > --- a/libstdc++-v3/testsuite/18_support/headers/csignal/macros.cc > +++ b/libstdc++-v3/testsuite/18_support/headers/csignal/macros.cc > @@ -37,6 +37,10 @@ namespace gnu > #error "SIG_DFL_must_be_a_macro" > #endif > > +#ifndef SIG_IGN > + #error "SIG_IGN_must_be_a_macro" > +#endif > + > #ifndef SIGFPE > #error "SIGFPE_must_be_a_macro" > #endif
Thanks, I'll apply this. I have no idea why we use underscores in those strings, but it's consistent with the rest of the file.
