>> Is this parenthesization correct ? >> >> OLD >> >> #define btokup(addr) (&kmemusage[(caddr_t)(addr) - kmembase >> >> PAGE_SHIFT]) >> >> NEW >> >> #define btokup(addr) (&kmemusage[((caddr_t)(addr) - kmembase) >> >> PAGE_SHIFT])
Yes. It is the same as before the "cleanup" in rev.1.13. >The added parentheses don't make any difference, semantically. This >change probably wouldn't meet the criteria spelled out in style(9): > > Unary operators don't require spaces, binary operators do. 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; > k = !(l & FLAGS); Nah, style(9), not to mention the example of btokup() in Lite1 and Lite2, requires paretheses here (in btokup(), and probably in the bad example in style(9), because the expression would be really confusing without them :-). Bruce To Unsubscribe: send mail to majord...@freebsd.org with "unsubscribe freebsd-current" in the body of the message
