On Fri, Dec 3, 2010 at 2:42 PM, David Hyatt <hy...@apple.com> wrote: > An example of nice boolean usage is: > > paintTextWithShadows(context, font, textRun, 0, length, length, > textOrigin, boxRect, textShadow, textStrokeWidth > 0, isHorizontal()); > GOOD! > > The last parameter is a boolean indicating whether or not the text run is > vertical text or horizontal text. It would be unreadable if you passed raw > true or raw false to the function, but all the call sites pass > isHorizontal(), so it's perfectly fine. >
Furthermore, the second-to-last arg here looks like a bool too, but "textStrokeWidth > 0" seems a lot more obvious to a reader than "true". PK
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev