[cfe-commits] If statement pretty printing

[Ben Lickly]

Ok, that seems reasonable.  Here's the updated patch.
~Ben

On Fri, Feb 6, 2009 at 3:32 PM, Sebastian Redl
<sebastian.redl at getdesigned.at> wrote:
> Ben Lickly wrote:
>> Is there any reason why the StmtPrinter.cpp doesn't put parenthesis
>> around the condition of an if statement when using -ast-print?  If
>> not, here is a patch to add them in.
>>
> Good catch, but I think the closing parenthesis should be written
> outside the if. It's just cleaner, and the pretty printer isn't that
> performance-sensitive.
>
> Sebastian
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ifstmt-pretty-printing-parens.patch
Type: application/octet-stream
Size: 774 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20090207/9442dacc/attachment.obj>