diff options
author | Joseph Myers <joseph@codesourcery.com> | 2017-03-16 13:31:57 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2017-03-16 13:31:57 +0000 |
commit | 2072f5c34ede81dde9e0d953d57a0690a66b0f12 (patch) | |
tree | 0833e169e1c3e9b24ef64e9010f7bb1e862f7da8 /time/bits | |
parent | d6fc3f6516cd20f195758086fbbbe3f17a8a6d95 (diff) | |
download | glibc-2072f5c34ede81dde9e0d953d57a0690a66b0f12.tar.gz glibc-2072f5c34ede81dde9e0d953d57a0690a66b0f12.tar.xz glibc-2072f5c34ede81dde9e0d953d57a0690a66b0f12.zip |
Remove C++ namespace handling from glibc headers.
glibc headers include some code (not particularly consistent or systematic) to put various declarations in C++ namespaces std and __c99, if _GLIBCPP_USE_NAMESPACES is defined. As noted in <https://gcc.gnu.org/ml/libstdc++/2017-03/msg00025.html>, this macro was removed from libstdc++ in 2000. I don't expect compilation with such old versions of libstdc++ to work with current glibc headers anyway (whereas old *binaries* are expected to stay working with current glibc); this patch (which should be a no-op with any libstdc++ version postdating that removal) removes all this code from the glibc headers. The begin-end-check.pl test, whose comments say it is about checking these namespace macro calls, is also removed. The code in that test would have covered __BEGIN_DECLS / __END_DECLS as well, but if those weren't properly matched it would show up with the check-installed-headers-cxx tests, so I don't think there is an actual use for keeping begin-end-check.pl with the namespace code removed. Tested for x86_64 and x86 (testsuite, and that installed stripped shared libraries are unchanged by the patch). * misc/sys/cdefs.h (__BEGIN_NAMESPACE_STD): Remove macro. (__END_NAMESPACE_STD): Likewise. (__USING_NAMESPACE_STD): Likewise. (__BEGIN_NAMESPACE_C99): Likewise. (__END_NAMESPACE_C99): Likewise. (__USING_NAMESPACE_C99): Likewise. * math/math.h (_Mdouble_BEGIN_NAMESPACE): Do not define and undefine macro. (_Mdouble_END_NAMESPACE): Likewise. * ctype/ctype.h: Do not handle C++ namespaces. * libio/bits/stdio-ldbl.h: Likewise. * libio/stdio.h: Likewise. * locale/locale.h: Likewise. * math/bits/mathcalls.h: Likewise. * setjmp/setjmp.h: Likewise. * signal/signal.h: Likewise. * stdlib/bits/stdlib-float.h: Likewise. * stdlib/bits/stdlib-ldbl.h: Likewise. * stdlib/stdlib.h: Likewise. * string/string.h: Likewise. * sysdeps/x86/fpu/bits/mathinline.h: Likewise. * time/bits/types/clock_t.h: Likewise. * time/bits/types/struct_tm.h: Likewise. * time/bits/types/time_t.h: Likewise. * time/time.h: Likewise. * wcsmbs/bits/wchar-ldbl.h: Likewise. * wcsmbs/uchar.h: Likewise. * wcsmbs/wchar.h: Likewise. [_GLIBCPP_USE_NAMESPACES] (wint_t): Remove conditional definition. * wctype/wctype.h: Do not handle C++ namespaces. * scripts/begin-end-check.pl: Remove. * Makefile (installed-headers): Likewise. (tests-special): Do not add $(objpfx)begin-end-check.out. ($(objpfx)begin-end-check.out): Remove.
Diffstat (limited to 'time/bits')
-rw-r--r-- | time/bits/types/clock_t.h | 6 | ||||
-rw-r--r-- | time/bits/types/struct_tm.h | 5 | ||||
-rw-r--r-- | time/bits/types/time_t.h | 5 |
3 files changed, 0 insertions, 16 deletions
diff --git a/time/bits/types/clock_t.h b/time/bits/types/clock_t.h index b8cbaf2cb6..ce97248f88 100644 --- a/time/bits/types/clock_t.h +++ b/time/bits/types/clock_t.h @@ -3,13 +3,7 @@ #include <bits/types.h> -__BEGIN_NAMESPACE_STD /* Returned by `clock'. */ typedef __clock_t clock_t; -__END_NAMESPACE_STD - -#if defined __USE_XOPEN || defined __USE_POSIX -__USING_NAMESPACE_STD(clock_t) -#endif #endif diff --git a/time/bits/types/struct_tm.h b/time/bits/types/struct_tm.h index 8f5af16715..b13b631228 100644 --- a/time/bits/types/struct_tm.h +++ b/time/bits/types/struct_tm.h @@ -4,7 +4,6 @@ #include <bits/types.h> /* ISO C `broken-down time' structure. */ -__BEGIN_NAMESPACE_STD struct tm { int tm_sec; /* Seconds. [0-60] (1 leap second) */ @@ -25,9 +24,5 @@ struct tm const char *__tm_zone; /* Timezone abbreviation. */ # endif }; -__END_NAMESPACE_STD -#if defined __USE_XOPEN || defined __USE_POSIX -__USING_NAMESPACE_STD(tm) -#endif #endif diff --git a/time/bits/types/time_t.h b/time/bits/types/time_t.h index 16e5269184..ab8287c6fe 100644 --- a/time/bits/types/time_t.h +++ b/time/bits/types/time_t.h @@ -3,12 +3,7 @@ #include <bits/types.h> -__BEGIN_NAMESPACE_STD /* Returned by `time'. */ typedef __time_t time_t; -__END_NAMESPACE_STD -#ifdef __USE_POSIX -__USING_NAMESPACE_STD(time_t) -#endif #endif |