[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