[cfe-commits] r172789 - in /cfe/trunk: lib/Format/Format.cpp lib/Format/UnwrappedLineParser.cpp unittests/Format/FormatTest.cpp

Jordan Rose jordan_rose at apple.com
Fri Jan 18 09:07:28 PST 2013


I haven't seen this style before, but LLVM convention is usually to put the break inside the braces anyway, so...


On Jan 17, 2013, at 21:50 , Nico Weber <nicolasweber at gmx.de> wrote:

> Author: nico
> Date: Thu Jan 17 23:50:57 2013
> New Revision: 172789
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=172789&view=rev
> Log:
> Formatter: After case blocks, "break" goes on the same line as the "}", PR14907.
> 
> Before:
> switch (foo) {
> case a: {
>  int a = g();
>  h(a);
> }
>  break;
> }
> 
> Now:
> switch (foo) {
> case a: {
>  int a = g();
>  h(a);
> } break;
> }

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130118/081e55c1/attachment.html>


More information about the cfe-commits mailing list