[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a781481a0705301223k1b52049eo33606cc158b00222@mail.gmail.com>
Date: Thu, 31 May 2007 00:53:20 +0530
From: "Satyam Sharma" <satyam.sharma@...il.com>
To: "Satyam Sharma" <satyam.sharma@...il.com>,
"Roland Dreier" <rdreier@...co.com>, openib-general@...nib.org,
linux-kernel@...r.kernel.org
Subject: Re: dealing with gcc 'comparison is always false' warnings (was: [PATCH] drivers/infiniband: fix comparsion between unsigned and negative)
Hi Bill,
On 5/31/07, Bill Nottingham <notting@...hat.com> wrote:
> Satyam Sharma (satyam.sharma@...il.com) said:
> > But yes, the kind of "fixes" you pointed out that _remove_ these conditions
> > are definitely *not* what we would want to do.
>
> I can see that - but I think it should be at least be brought up for each
> warning, to determine either:
>
> 1) if it should be ignored
> 2) if a signed type is actually intended
> 3) if the code should be elided
Agreed. The extract you've pointed out above was too strongly worded
unnecessarily / wrong generalization, and I corrected it later.
> While not necessarily in the IB instances, there are cases where there
> are entire blocks of code (with debugging output, error returns, etc)
> that can never get run, and it may make sense to remove those.
Agreed, again.
Satyam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists