diff mbox series

[committed] Fix several preprocessor directives

Message ID 2c5a8abd-c03e-294f-74bc-9e3f3f1de393@gmail.com
State New
Headers show
Series [committed] Fix several preprocessor directives | expand

Commit Message

François Dumont July 30, 2023, 1:55 p.m. UTC
Committed as obvious.

     libstdc++: Fix several preprocessor directives

     A wrong usage of #define in place of a #error seems to have been 
replicated
     at different places in source files.

     libstdc++-v3/ChangeLog:

             * src/c++11/compatibility-ldbl-facets-aliases.h: Replace 
#define with
             proper #error.
             * src/c++11/locale-inst-monetary.h: Likewise.
             * src/c++11/locale-inst-numeric.h: Likewise.

François

Comments

Jonathan Wakely Aug. 7, 2023, 10:46 a.m. UTC | #1
On Sun, 30 Jul 2023 at 14:55, François Dumont via Libstdc++ <
libstdc++@gcc.gnu.org> wrote:

> Committed as obvious.
>
>      libstdc++: Fix several preprocessor directives
>
>      A wrong usage of #define in place of a #error seems to have been
> replicated
>      at different places in source files.
>

Oops, thanks!


>
>      libstdc++-v3/ChangeLog:
>
>              * src/c++11/compatibility-ldbl-facets-aliases.h: Replace
> #define with
>              proper #error.
>              * src/c++11/locale-inst-monetary.h: Likewise.
>              * src/c++11/locale-inst-numeric.h: Likewise.
>
> François
>
diff mbox series

Patch

diff --git a/libstdc++-v3/src/c++11/compatibility-ldbl-facets-aliases.h b/libstdc++-v3/src/c++11/compatibility-ldbl-facets-aliases.h
index 70c9342d88a..faf8221b273 100644
--- a/libstdc++-v3/src/c++11/compatibility-ldbl-facets-aliases.h
+++ b/libstdc++-v3/src/c++11/compatibility-ldbl-facets-aliases.h
@@ -23,11 +23,11 @@ 
 // <http://www.gnu.org/licenses/>.
 
 #ifndef C
-#define "This file should not be compiled directly, only included"
+# error "This file should not be compiled directly, only included"
 #endif
 
 #ifndef _GLIBCXX_LONG_DOUBLE_COMPAT
-#define "This file should only be used for _GLIBCXX_LONG_DOUBLE_COMPAT builds"
+# error "This file should only be used for _GLIBCXX_LONG_DOUBLE_COMPAT builds"
 #endif
 
 // XXX GLIBCXX_ABI Deprecated
diff --git a/libstdc++-v3/src/c++11/locale-inst-monetary.h b/libstdc++-v3/src/c++11/locale-inst-monetary.h
index d8fecf26596..954de1f52cf 100644
--- a/libstdc++-v3/src/c++11/locale-inst-monetary.h
+++ b/libstdc++-v3/src/c++11/locale-inst-monetary.h
@@ -23,7 +23,7 @@ 
 // <http://www.gnu.org/licenses/>.
 
 #ifndef C
-#define "This file should not be compiled directly, only included"
+# error "This file should not be compiled directly, only included"
 #endif
 
 #include "facet_inst_macros.h"
diff --git a/libstdc++-v3/src/c++11/locale-inst-numeric.h b/libstdc++-v3/src/c++11/locale-inst-numeric.h
index c77ee9e8d38..b917fe5802e 100644
--- a/libstdc++-v3/src/c++11/locale-inst-numeric.h
+++ b/libstdc++-v3/src/c++11/locale-inst-numeric.h
@@ -23,7 +23,7 @@ 
 // <http://www.gnu.org/licenses/>.
 
 #ifndef C
-#define "This file should not be compiled directly, only included"
+# error "This file should not be compiled directly, only included"
 #endif
 
 #include "facet_inst_macros.h"