svn commit: r334669 - head/sys/sys
Andriy Gapon
avg at FreeBSD.org
Tue Jun 5 22:16:00 UTC 2018
On 06/06/2018 00:07, Eric van Gyzen wrote:
> On 06/05/2018 15:53, Ian Lepore wrote:
>> This is a sad sad thing. Treating (bits & flagconstants) as boolean has
>> a long long history in C. Surely there are literally thousand of
>> occurrances in freebsd code already, so why did this one get flagged?
>
> I agree, and I tend to avoid adding "!= 0" unnecessarily, but I don't
> feel very strongly about it.
I am not sure what parts of code you work on, but turning bit operations into
logical operations within conditions is the predominant FreeBSD style.
Please accept it.
--
Andriy Gapon
More information about the svn-src-all
mailing list