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
