Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 12 Jan 1999 14:35:51 +0100
From:      Eivind Eklund <eivind@FreeBSD.ORG>
To:        Luigi Rizzo <luigi@labinfo.iet.unipi.it>
Cc:        cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/sys/netinet ip_input.c
Message-ID:  <19990112143551.D65459@bitbox.follo.net>
In-Reply-To: <199901121117.MAA23744@labinfo.iet.unipi.it>; from Luigi Rizzo on Tue, Jan 12, 1999 at 12:17:22PM %2B0100
References:  <199901121225.EAA12902@freefall.freebsd.org> <199901121117.MAA23744@labinfo.iet.unipi.it>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jan 12, 1999 at 12:17:22PM +0100, Luigi Rizzo wrote:
> >   Modified files:
> >     sys/netinet          ip_input.c 
> >   Log:
> >   Add #ifdef's to avoid unused label warning in some cases.
> 
> i wonder if it is preferable to clutter the code with #ifdef's or
> tolerate the potential warnings.

Unless good counter-arguments spring forward, -Werror is going in
shortly, so tolerating the warnings is not an option.

This is because people otherwise ignore their warnings, and this make
it difficult to use the warnings to remove true bugs (of which I've
found at least 3 during this round of commits).

Eivind.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message



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