[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