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

[jahanian]

The new look looks weird.

- Fariborz

On Jan 18, 2013, at 9:07 AM, Jordan Rose <jordan_rose at apple.com> wrote:

> 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;
>> }
> 
> _______________________________________________
> 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/20130118/5f9fc6e8/attachment.html>