r293675 - Keep Chromium ObjC column limit at 80 for consistency with C++

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 31 10:42:06 PST 2017


Author: nico
Date: Tue Jan 31 12:42:05 2017
New Revision: 293675

URL: http://llvm.org/viewvc/llvm-project?rev=293675&view=rev
Log:
Keep Chromium ObjC column limit at 80 for consistency with C++

https://reviews.llvm.org/D29337
Patch from Dan Beam <dbeam at chromium.org>!

Modified:
    cfe/trunk/lib/Format/Format.cpp

Modified: cfe/trunk/lib/Format/Format.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/Format.cpp?rev=293675&r1=293674&r2=293675&view=diff
==============================================================================
--- cfe/trunk/lib/Format/Format.cpp (original)
+++ cfe/trunk/lib/Format/Format.cpp Tue Jan 31 12:42:05 2017
@@ -653,6 +653,8 @@ FormatStyle getChromiumStyle(FormatStyle
     ChromiumStyle.AllowShortLoopsOnASingleLine = false;
     ChromiumStyle.BinPackParameters = false;
     ChromiumStyle.DerivePointerAlignment = false;
+    if (Language == FormatStyle::LK_ObjC)
+      ChromiumStyle.ColumnLimit = 80;
   }
   ChromiumStyle.SortIncludes = false;
   return ChromiumStyle;




More information about the cfe-commits mailing list