This is the mail archive of the libc-hacker@sourceware.cygnus.com mailing list for the glibc project.

Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Some small fixes



The flags in intl aren't needed anymore (according to my tests).  For
the math files I've removed -Wno-write-strings and use it only where
it's really needed.

I'm also appending patches to avoid the following warnings on alpha:
td_ta_thr_iter.c:96: warning: type mismatch in implicit declaration for built-in function `alloca'
td_ta_tsd_iter.c:39: warning: type mismatch in implicit declaration for built-in function `alloca'
../sysdeps/alpha/fpu/fraiseexcpt.c:66: warning: `return' with a value, in function returning void

Btw. regex.c outputs a number of warnings on 64bit platforms like:
regex.c:686: warning: int format, different type arg (arg 2)

For this code (p and start are pointers):
      printf ("%d:\t", p - start);

We could use either %ld or %t here - or cast the result to an int.
What's the best way to fix this?

Andreas

For linuxthreads_db/ChangeLog:
2000-02-25  Andreas Jaeger  <aj@suse.de>

	* td_ta_thr_iter.c: Include <alloca.h> for prototype declaration.
	* td_ta_tsd_iter.c: Likewise.

For the main ChangeLog:

2000-02-25  Andreas Jaeger  <aj@suse.de>

	* sysdeps/alpha/fpu/fraiseexcpt.c (__feraiseexcept): Correct
	declaration to return value.

	* sysdeps/ieee754/Makefile (CFLAGS-k_standard.c): Added.

	* math/Makefile: Remove -Wno-write-strings since it's only needed
	by one file.

	* intl/Makefile (CFLAGS-bindtextdom.c): The code has been fixed,
	we don't need to disable the warnings anymore, remove them.
	(CFLAGS-finddomain.c): Likewise.
	(CFLAGS-localealias.c): Likewise.


============================================================
Index: sysdeps/alpha/fpu/fraiseexcpt.c
--- sysdeps/alpha/fpu/fraiseexcpt.c	1999/10/31 23:13:21	1.4
+++ sysdeps/alpha/fpu/fraiseexcpt.c	2000/02/25 20:06:35
@@ -1,5 +1,5 @@
 /* Raise given exceptions.
-   Copyright (C) 1997, 1998, 1999 Free Software Foundation, Inc.
+   Copyright (C) 1997, 1998, 1999, 2000  Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Richard Henderson <rth@tamu.edu>, 1997.
 
@@ -21,7 +21,7 @@
 #include <fenv.h>
 #include <math.h>
 
-void
+int
 __feraiseexcept (int excepts)
 {
   double tmp;
============================================================
Index: sysdeps/ieee754/Makefile
--- sysdeps/ieee754/Makefile	1997/01/01 15:27:27	1.1
+++ sysdeps/ieee754/Makefile	2000/02/25 20:06:35
@@ -1,3 +1,5 @@
 ifeq ($(subdir),math)
 sysdep_headers += ieee754.h
+CFLAGS-k_standard.c = -Wno-write-strings
 endif
+
============================================================
Index: math/Makefile
--- math/Makefile	2000/01/27 01:46:45	1.93
+++ math/Makefile	2000/02/25 20:06:35
@@ -165,7 +165,7 @@
 endif
 
 # The fdlibm code generates a lot of these warnings but is otherwise clean.
-override CFLAGS += -Wno-uninitialized -Wno-write-strings
+override CFLAGS += -Wno-uninitialized
 
 # The -lieee library is actually an object file.
 # The module just defines the _LIB_VERSION_ variable.
============================================================
Index: intl/Makefile
--- intl/Makefile	2000/02/11 18:39:35	1.20
+++ intl/Makefile	2000/02/25 20:06:36
@@ -86,6 +86,3 @@
 
 endif
 
-CFLAGS-bindtextdom.c = -Wno-unused
-CFLAGS-finddomain.c = -Wno-unused
-CFLAGS-localealias.c = -Wno-unused
============================================================
Index: linuxthreads_db/td_ta_thr_iter.c
--- linuxthreads_db/td_ta_thr_iter.c	1999/11/12 23:28:15	1.9
+++ linuxthreads_db/td_ta_thr_iter.c	2000/02/25 20:06:36
@@ -1,5 +1,5 @@
 /* Iterate over a process's threads.
-   Copyright (C) 1999 Free Software Foundation, Inc.
+   Copyright (C) 1999, 2000 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@cygnus.com>, 1999.
 
@@ -19,7 +19,7 @@
    Boston, MA 02111-1307, USA.  */
 
 #include "thread_dbP.h"
-
+#include <alloca.h>
 
 static int
 handle_descr (const td_thragent_t *ta, td_thr_iter_f *callback,
============================================================
Index: linuxthreads_db/td_ta_tsd_iter.c
--- linuxthreads_db/td_ta_tsd_iter.c	1999/11/04 06:46:14	1.2
+++ linuxthreads_db/td_ta_tsd_iter.c	2000/02/25 20:06:36
@@ -1,5 +1,5 @@
 /* Iterate over a process's thread-specific data.
-   Copyright (C) 1999 Free Software Foundation, Inc.
+   Copyright (C) 1999, 2000 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@cygnus.com>, 1999.
 
@@ -19,7 +19,7 @@
    Boston, MA 02111-1307, USA.  */
 
 #include "thread_dbP.h"
-
+#include <alloca.h>
 
 td_err_e
 td_ta_tsd_iter (const td_thragent_t *ta, td_key_iter_f *callback,
-- 
 Andreas Jaeger
  SuSE Labs aj@suse.de
   private aj@arthur.rhein-neckar.de

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]