Bruce Evans wrote:
> On Thu, 1 Oct 2009, [utf-8] Dag-Erling Smørgrav wrote:
>
>> Coleman Kane <[email protected]> writes:
>>> - if (sc->ndis_80211 && vap)
>>> + if ((sc->ndis_80211 != NULL) && (vap != NULL))
>>
>> sc->ndis_80211 is an int. NULL is a pointer.
>
> Also, the number of style bugs was doubled on (almost?) every changed line
> by adding 2 sets of unnecessary parentheses.
FWIW I simply asked he compare ptr's against NULL. The excessive
paren's hurt my eyes too :-)
Sam
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"