Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 25 Apr 1996 04:05:25 +0930 (CST)
From:      Michael Smith <msmith@atrad.adelaide.edu.au>
To:        jkh@time.cdrom.com (Jordan K. Hubbard)
Cc:        msmith@atrad.adelaide.edu.au, jmacd@CS.Berkeley.EDU, bde@zeta.org.au, freebsd-hackers@FreeBSD.org
Subject:   Re: GNU binutils port
Message-ID:  <199604241835.EAA19918@genesis.atrad.adelaide.edu.au>
In-Reply-To: <11585.830359686@time.cdrom.com> from "Jordan K. Hubbard" at Apr 24, 96 08:28:06 am

next in thread | previous in thread | raw e-mail | index | archive | help
Jordan K. Hubbard stands accused of saying:
> 
> I use -Wall in order to help me find stupid bugs in my code.  It's
> actually pretty good at this, and so I use it.  In order to use it, I
> also add things like extra parentheses around assignment expressions
> (and, given that I also happen to *prefer* the: ``if ((blah = frob())
> != bar)'' style, that's scarcely a hardship) and basically do whatever
> else I need to do to make the fool thing happy.  That's just part of
> the price for using the tool, same as with any tool.

...this is exactly why I use -Wall for my code, and follow similar
conventions.  However, not everyone does.  Pointing at a (large)
piece of software that generates a spastic number of warnings with 
-Wall and saying "this software is broken" isn't terribly valid.

Having said that, I seem to remember that one of the oldest comments
in the 'things for newcomers to do' file has been "make the
kernel compile cleanly with -Wall" 8)

> 						Jordan

-- 
]] Mike Smith, Software Engineer        msmith@atrad.adelaide.edu.au    [[
]] Genesis Software                     genesis@atrad.adelaide.edu.au   [[
]] High-speed data acquisition and      (GSM mobile) 0411-222-496       [[
]] realtime instrument control          (ph/fax)  +61-8-267-3039        [[
]] Collector of old Unix hardware.      "Where are your PEZ?" The Tick  [[



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199604241835.EAA19918>