[PATCH] D148437: [clang-format] Dont interpret variable named interface as keyword for C++
Jorge Pinto Sousa via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Apr 15 15:56:46 PDT 2023
sousajo updated this revision to Diff 513946.
sousajo added a comment.
- inline string to be checked
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148437/new/
https://reviews.llvm.org/D148437
Files:
clang/lib/Format/UnwrappedLineParser.cpp
clang/unittests/Format/FormatTest.cpp
Index: clang/unittests/Format/FormatTest.cpp
===================================================================
--- clang/unittests/Format/FormatTest.cpp
+++ clang/unittests/Format/FormatTest.cpp
@@ -25429,6 +25429,13 @@
verifyFormat("auto x = 5s .count() == 5;");
}
+TEST_F(FormatTest, InterfaceAsClassMemberName) {
+ verifyFormat("class Foo {\n"
+ " int interface;\n"
+ " Foo::Foo(int iface) : interface{iface} {}\n"
+ "}");
+}
+
} // namespace
} // namespace test
} // namespace format
Index: clang/lib/Format/UnwrappedLineParser.cpp
===================================================================
--- clang/lib/Format/UnwrappedLineParser.cpp
+++ clang/lib/Format/UnwrappedLineParser.cpp
@@ -1881,7 +1881,7 @@
}
}
- if (FormatTok->is(Keywords.kw_interface)) {
+ if (!Style.isCpp() && FormatTok->is(Keywords.kw_interface)) {
if (parseStructLike())
return;
break;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148437.513946.patch
Type: text/x-patch
Size: 981 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230415/f6526603/attachment.bin>
More information about the cfe-commits
mailing list