Ethereal-dev: [Ethereal-dev] Patch: Turn on warnings [Was: More signed/unsigned (wiretap)] (fo

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Thu, 21 Jun 2001 14:19:09 +0200
Hello,

looks like I replied to Guy personally instread of the list...

  Ciao
        Jï¿œrg

----- Forwarded message from Joerg Mayer <jmayer@xxxxxxxxx> -----

On Wed, Jun 20, 2001 at 12:10:17PM -0700, Guy Harris wrote:
> Will you also eventually submit the patch that fixes the warnings? :-) 
> (The patch wasn't attached to your message.)

Yes, I only remembered that I had forgotten to submit it. I've just
generated it so here we go (see attach).

  Ciao
      Jï¿œrg
--
Joerg Mayer                                          <jmayer@xxxxxxxxx>
I found out that "pro" means "instead of" (as in proconsul). Now I know
what proactive means.

Changelog: Jï¿œrg Mayer
	Turn on additional warnings for wiretap and epan too.

Index: ethereal/wiretap/configure.in
===================================================================
RCS file: /cvsroot/ethereal/wiretap/configure.in,v
retrieving revision 1.26
diff -u -r1.26 configure.in
--- configure.in	2001/04/17 22:47:37	1.26
+++ configure.in	2001/06/20 19:16:43
@@ -29,7 +29,7 @@
 # If we're running gcc, add '-Wall' to CFLAGS.
 AC_MSG_CHECKING(to see if we can add '-Wall' to CFLAGS)
 if test x$GCC != x ; then
-  CFLAGS="-Wall $CFLAGS"
+  CFLAGS="-Wall -W $CFLAGS"
   AC_MSG_RESULT(yes)
 else
   AC_MSG_RESULT(no)
Index: ethereal/epan/configure.in
===================================================================
RCS file: /cvsroot/ethereal/epan/configure.in,v
retrieving revision 1.8
diff -u -r1.8 configure.in
--- configure.in	2001/05/16 16:17:52	1.8
+++ configure.in	2001/06/20 19:16:44
@@ -29,7 +29,7 @@
 # If we're running gcc, add '-Wall' to CFLAGS.
 AC_MSG_CHECKING(to see if we can add '-Wall' to CFLAGS)
 if test x$GCC != x ; then
-  CFLAGS="-Wall $CFLAGS"
+  CFLAGS="-Wall -W $CFLAGS"
   AC_MSG_RESULT(yes)
 else
   AC_MSG_RESULT(no)


----- End forwarded message -----

--
Joerg Mayer                                          <jmayer@xxxxxxxxx>
I found out that "pro" means "instead of" (as in proconsul). Now I know
what proactive means.