>Anyway, if we're going to -Wall'ify the kernel (as we should) >then we need to update sytle(9) to reflect that. > >In fact, style(9) should say: > > If at all possible, your code should compile without warnings > when the gcc -Wall flag is given.
Avoiding warnings is more an engineering than a stylistic matter. You turn on warnings to help avoid bugs that the compiler can find easily. You ask everyone else to turn on warnings so that compiling their sources with the same CFLAGS as your sources doesn't cause a spew of warnings. >As it stands now (and I QUOTE!) it says: > > Don't use parentheses unless they're required for precedence, or > the statement is really confusing without them. > > a = b->c[0] + ~d == (e || f) || g && h ? i : j >> 1; > >That's ridiculous! I think it's trying to be funny. This makes it a bad example either way. Perhaps its point is that complicated expressions can't be made less confusing by adding parentheses. Bruce To Unsubscribe: send mail to majord...@freebsd.org with "unsubscribe freebsd-current" in the body of the message