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 <federic...@gmail.com> wrote: > 2011/10/21 Anthony Petrov <anthony.pet...@oracle.com>: >> >> 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. >
