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

[Thomas Gibson-Robinson]

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/ef1f6ffe/attachment.html>