[cfe-commits] If statement pretty printing

Sebastian Redl sebastian.redl at getdesigned.at
Sat Feb 7 10:54:42 PST 2009


Ben Lickly wrote:
> Ok, that seems reasonable.  Here's the updated patch.
>   
Thanks, I'll commit it as soon as I've got a clean tree.

Sebastian
>
> 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
>>
>>     




More information about the cfe-commits mailing list