about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2017-08-07 19:53:17 +0000
committerJoseph Myers <joseph@codesourcery.com>2017-08-07 19:53:17 +0000
commit8b1647877c7f797161dfe1a6b35eab8534d98f9f (patch)
tree5ef137d0791435214795a0d889599b3ec971d6fd
parentdd3e86ad7cf5ea1b5a42b7962732f98882c63ff8 (diff)
downloadglibc-8b1647877c7f797161dfe1a6b35eab8534d98f9f.tar.gz
glibc-8b1647877c7f797161dfe1a6b35eab8534d98f9f.tar.xz
glibc-8b1647877c7f797161dfe1a6b35eab8534d98f9f.zip
Remove __long_double_t.
sys/cdefs.h has a macro __long_double_t used in two places in glibc.
long double is a standard part of C since C89; there is no need for
such an alias for it.  This patch removes that macro and uses long
double directly everywhere.  As an implementation-namespace,
undocumented symbol, it should not be considered part of the API for
users, and codesearch.debian.net shows no sign of it being used
outside glibc in a way that would break with this patch.

Tested for x86_64.

	* misc/sys/cdefs.h (__long_double_t): Remove.
	* stdio-common/printf_fp.c (__printf_fp_l): Use long double
	instead of __long_double_t,
	* stdlib/strfmon_l.c (__vstrfmon_l): Likewise.
-rw-r--r--ChangeLog7
-rw-r--r--misc/sys/cdefs.h1
-rw-r--r--stdio-common/printf_fp.c2
-rw-r--r--stdlib/strfmon_l.c2
4 files changed, 9 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index b64ed0571d..05a8d8eab0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2017-08-07  Joseph Myers  <joseph@codesourcery.com>
+
+	* misc/sys/cdefs.h (__long_double_t): Remove.
+	* stdio-common/printf_fp.c (__printf_fp_l): Use long double
+	instead of __long_double_t,
+	* stdlib/strfmon_l.c (__vstrfmon_l): Likewise.
+
 2017-08-07  Siddhesh Poyarekar  <siddhesh@sourceware.org>
 
 	* benchtests/scripts/compare_strings.py: Avoid display error when
diff --git a/misc/sys/cdefs.h b/misc/sys/cdefs.h
index 06523bfe9c..b3e7f3be96 100644
--- a/misc/sys/cdefs.h
+++ b/misc/sys/cdefs.h
@@ -102,7 +102,6 @@
 
 /* This is not a typedef so `const __ptr_t' does the right thing.  */
 #define __ptr_t void *
-#define __long_double_t  long double
 
 
 /* C++ needs to know that types and declarations are C, not C++.  */
diff --git a/stdio-common/printf_fp.c b/stdio-common/printf_fp.c
index 514b698d27..3ed4037ba5 100644
--- a/stdio-common/printf_fp.c
+++ b/stdio-common/printf_fp.c
@@ -217,7 +217,7 @@ __printf_fp_l (FILE *fp, locale_t loc,
   union
     {
       double dbl;
-      __long_double_t ldbl;
+      long double ldbl;
 #if __HAVE_DISTINCT_FLOAT128
       _Float128 f128;
 #endif
diff --git a/stdlib/strfmon_l.c b/stdlib/strfmon_l.c
index 69ca9d5d3b..98554dfe80 100644
--- a/stdlib/strfmon_l.c
+++ b/stdlib/strfmon_l.c
@@ -100,7 +100,7 @@ __vstrfmon_l (char *s, size_t maxsize, locale_t loc, const char *format,
       union
       {
 	double dbl;
-	__long_double_t ldbl;
+	long double ldbl;
       }
       fpnum;
       int int_format;