On Wed, 2013-04-10 at 15:57 -0700, Andrew Morton wrote: > On Tue, 09 Apr 2013 20:17:14 -0700 Joe Perches <j...@perches.com> wrote: > > Comparisons of A to true and false are better written > > as A and !A. > > Bleat a message on use. > hm. I'm counting around 1,100 instances of "== true" and "== false".
And about all of them in are staging, where I think they really should be fixed. $ find . -maxdepth 2 -type d \ while read file ; do \ echo "$(git grep -E '(==|\!=)\s*(true|false)' $file | wc -l) $file"; \ done | sort -rn | head -10 1375 . 1298 ./drivers 1055 ./drivers/staging 63 ./drivers/net 59 ./drivers/gpu 24 ./net 20 ./drivers/media 17 ./net/nfc 13 ./fs 11 ./drivers/usb > That's a lot of people to shout at. Not really. > Is it really worthwhile? I think so. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/