Re: [PATCH] checkpatch: Warn on comparisons to true and false
From: Joe Perches
Date: Thu Apr 11 2013 - 04:29:44 EST
On Thu, 2013-04-11 at 11:19 +0300, Dan Carpenter wrote:
> On Wed, Apr 10, 2013 at 10:14:15PM -0400, Dave Jones wrote:
> > It would be interesting to see how many people have historically screwed
> > up and used (!a) when they mean (a) and vice versa, versus spelling
> > it out longform. I'd be surprised if the results weren't skewed
> > in favour of the more verbose form.
> I see a the occasional reversed test in Smatch but normally these
> kind of bugs are detected with basic testing so they are rare.
I'd guess the most common error would be using
an int comparison when the value is not 0 or 1.
Non-zero is still "true" but isn't == true.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/