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

[Thomas Gibson-Robinson]

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