[lmi-commits] [lmi] master a0ad509 2/2: Don't undefine __STRICT_ANSI__ before including libstdc++ headers

[Greg Chicares]

branch: master
commit a0ad50975c787caccf36e1d341010fe96a1cf28d
Author: Vadim Zeitlin <vadim@tt-solutions.com>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Don't undefine __STRICT_ANSI__ before including libstdc++ headers
    
    This is not supported with the libstdc++ version used by gcc 11 and
    results in errors similar to the following:
    
    /usr/include/x86_64-linux-gnu/c++/11/bits/c++config.h:558:2:
    error: #warning "__STRICT_ANSI__ seems to have been undefined; this is
    not supported" [-Werror=cpp]
    
    It also doesn't seem to be actually needed with the previous versions
    any longer, so just don't do it at all.
---
 platform_dependent.hpp | 40 ----------------------------------------
 1 file changed, 40 deletions(-)

diff --git a/platform_dependent.hpp b/platform_dependent.hpp
index 92ce801..376173d 100644
--- a/platform_dependent.hpp
+++ b/platform_dependent.hpp
@@ -24,35 +24,6 @@
 
 #include "config.hpp"
 
-// When compiling with '-ansi' or certain '-std=' options, gcc defines
-// macro __STRICT_ANSI__. Standard headers like <stdlib.h> may provide
-// prototypes for nonstandard functions like putenv() only if that
-// macro is not defined, in order to conform to C99 4/7. [Consider
-// the following strictly-conforming program (C99 4/6):
-//
-//   #include <stdlib.h>
-//   void foo(void);
-//   int main() {foo(); return 0;}
-//
-// and replace 'foo' with 'putenv', which latter name the language
-// standard clearly does not reserve.]
-//
-// However, ISO 9945:2002 (POSIX) declares putenv() in <stdlib.h>. In
-// light of the present header's raison d'être, it is sensible to
-// suspend the operation of the __STRICT_ANSI__ mechanism, taking care
-// to restore it after including other headers here.
-//
-// Some of the functions made accessible here provide capabilities
-// that are useful but absent from the standard language: for example,
-// it is difficult to implement cgi-bin without putenv(). Some others:
-//   _wcsdup(), fileno(), strcasecmp(), strdup()
-// should be avoided in general, but are required by wx.
-
-#if defined __GNUC__ && defined __STRICT_ANSI__
-#   define LMI_GNUC_STRICT_ANSI
-#   undef __STRICT_ANSI__
-#endif // defined __GNUC__ && defined __STRICT_ANSI__
-
 #if defined LMI_POSIX
 #   include <stdio.h>                   // fileno()
 #   include <stdlib.h>                  // putenv()
@@ -91,15 +62,4 @@
 // means, locally, wherever it's needed. As of 2017-04, getch() is no
 // longer used; this paragraph is kept lest it be reintroduced.
 
-// Although gcc may once have defined __STRICT_ANSI__ differently:
-//   http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3199
-// its intended value now is 1:
-//   http://gcc.gnu.org/onlinedocs/cpp/Common-Predefined-Macros.html
-
-#if defined LMI_GNUC_STRICT_ANSI
-#   define __STRICT_ANSI__ 1
-#endif // defined LMI_GNUC_STRICT_ANSI
-
-#undef LMI_GNUC_STRICT_ANSI
-
 #endif // platform_dependent_hpp