[PATCH] D44632: [clang-format] Add a few more Core Graphics identifiers to ObjC heuristic

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 20 02:40:34 PDT 2018


djasper added inline comments.


================
Comment at: lib/Format/Format.cpp:1450
     // Keep this array sorted, since we are binary searching over it.
     static constexpr llvm::StringLiteral FoundationIdentifiers[] = {
         "CGFloat",
----------------
I have concerns about this growing lists of things. Specifically:
- Keeping it sorted is a maintenance concern.
- Doing binary search for basically every identifier in a header seems an unnecessary waste.

Could we just create a hash set of these?


================
Comment at: unittests/Format/FormatTest.cpp:12099
   EXPECT_EQ(FormatStyle::LK_ObjC, guessLanguage("foo", "@interface Foo\n at end\n"));
+  EXPECT_EQ(FormatStyle::LK_ObjC, guessLanguage("foo.h", "int DoStuff(CGRect rect);\n"));
+  EXPECT_EQ(
----------------
jolesiak wrote:
> I know that it's violated in several places in this file (even in two of the three lines above), but I feel like we should keep 80 char limit for column width.
Agreed. Please format this file with clang-format.


Repository:
  rC Clang

https://reviews.llvm.org/D44632





More information about the cfe-commits mailing list