James Carlson <[EMAIL PROTECTED]> wrote: > Alexey Dobriyan writes: >> - if (ap == 0) >> + if (!ap) > > And the solution is to treat it as a boolean instead?! I'm not sure > which is more ugly.
Treating it as a boolean looks good to me. It's better than the existing code because it shuts sparse up. -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
