On Thursday 27 April 2006 09:47, quozl at us.netrek.org wrote:
>   A future patch may add -Wall to system.mk.in after a few other problems
> are solved.

I suggest turning it on now.

Add -Werror after you fix the existing warnings - it will prevent (hopefully) 
people committing code which generates warnings.

If you're feeling really enthusiastic you could recompile it with..
-Wsystem-headers -Werror -Wall -Wno-format-y2k -W -Wno-unused-parameter 
-Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Wreturn-type 
-Wcast-qual -Wwrite-strings -Wswitch -Wshadow -Wcast-align -Wunused-parameter 
-Wchar-subscripts -Winline -Wnested-externs -Wredundant-decls

Although not much code survives that level of compiler paranoia :)

Other good tools are Electric Fence and Valgrind - they are run time tools 
though, but are *excellent* for finding buffer overruns and the like 
(especially Valgrind which also finds memory leaks)

-- 
Daniel O'Connor software and network engineer
for Genesis Software - http://www.gsoft.com.au
"The nice thing about standards is that there
are so many of them to choose from."
  -- Andrew Tanenbaum
GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C

-- 
Daniel O'Connor software and network engineer
for Genesis Software - http://www.gsoft.com.au
"The nice thing about standards is that there
are so many of them to choose from."
  -- Andrew Tanenbaum
GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mailman.us.netrek.org/pipermail/netrek-dev/attachments/20060427/c1cee2a4/attachment.pgp