[PATCH] clang-format: Fix some problems with Allman brace breaking

[Manuel Klimek]

Thx. Submitted as r187901


On Wed, Aug 7, 2013 at 8:09 AM, Thomas Gibson-Robinson <
thomas.gibsonrobinson at gmail.com> wrote:

> That would be great.
>
> Thanks!
>
>
> On 7 Aug 2013, at 16:08, Manuel Klimek <klimek at google.com> wrote:
>
> Do you need me to commit it?
>
>
> On Wed, Aug 7, 2013 at 12:20 AM, Thomas Gibson-Robinson <
> thomas.gibsonrobinson at gmail.com> wrote:
>
>> Sorry about that: next time I'll make sure to use phabricator.
>>
>> Thanks for looking at the patch!
>>
>> On 6 Aug 2013, at 18:50, Manuel Klimek <klimek at google.com> wrote:
>>
>> LG. In the future, it would be cool if you could use phabricator (
>> http://llvm-reviews.chandlerc.com/, docs at
>> http://llvm.org/docs/Phabricator.html), as that makes it easier to
>> review code in context.
>>
>> Thanks!
>>
>>
>>
>> On Mon, Aug 5, 2013 at 2:32 PM, Thomas Gibson-Robinson <
>> thomas.gibsonrobinson at gmail.com> wrote:
>>
>>> The attached patch fixes a couple of bugs with the Allman brace
>>> breaking. In particular, left braces after an enum declaration now occur on
>>> their own line. Further, when short ifs/whiles are allowed these no longer
>>> cause the left brace to be on the same line as the if/while when a brace is
>>> included. Tests are included.
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>>
>>>
>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130807/3e7706ea/attachment.html>