Thx. Submitted as r187901
On Wed, Aug 7, 2013 at 8:09 AM, Thomas Gibson-Robinson < [email protected]> wrote: > That would be great. > > Thanks! > > > On 7 Aug 2013, at 16:08, Manuel Klimek <[email protected]> wrote: > > Do you need me to commit it? > > > On Wed, Aug 7, 2013 at 12:20 AM, Thomas Gibson-Robinson < > [email protected]> 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 <[email protected]> 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 < >> [email protected]> 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 >>> [email protected] >>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >>> >>> >> >> > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
