Could you please use curly braces even for one-line ifs. It looks like it is a part of the style.
Regards, Oleg. On Sat, Oct 22, 2011 at 6:30 AM, Federico <[email protected]> wrote: > 2011/10/21 Anthony Petrov <[email protected]>: >> >> Could you make a patch, test it, and post it to this mailing list for review >> please? >> > > Here's a patch. I tried to follow the style used in other places where > one line if statements are used to avoid calling the logging code. > > I hope the patch is acceptable. >
