Hi,

On 3 April 2012 14:58, Tiago Vignatti <[email protected]> wrote:
> Just moving code around. No functional changes.

Sorry, I'm not sure I really like this one, unless you're planning to
add an else branch; even then, that might well be fodder for helper
functions.  The early-return style is good not only at reducing our
indentation levels -- which is pretty essential with eight-space
indents -- but also at making it immediately obvious what's going to
get returned for the uncommon case.

Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to