[PATCH] D93490: [clang-format] PR48539 ReflowComments breaks Qt translation comments
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 23 06:45:51 PST 2020
This revision was automatically updated to reflect the committed changes.
Closed by commit rG031743cb5b3c: [clang-format] PR48539 ReflowComments breaks Qt translation comments (authored by MyDeveloperDay).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93490/new/
https://reviews.llvm.org/D93490
Files:
clang/lib/Format/BreakableToken.cpp
clang/unittests/Format/FormatTestComments.cpp
Index: clang/unittests/Format/FormatTestComments.cpp
===================================================================
--- clang/unittests/Format/FormatTestComments.cpp
+++ clang/unittests/Format/FormatTestComments.cpp
@@ -702,6 +702,12 @@
" // long 1 2 3 4 5 6\n"
"}",
getLLVMStyleWithColumns(20)));
+
+ EXPECT_EQ("//: A comment that\n"
+ "//: doesn't fit on\n"
+ "//: one line",
+ format("//: A comment that doesn't fit on one line",
+ getLLVMStyleWithColumns(20)));
}
TEST_F(FormatTestComments, PreservesHangingIndentInCxxComments) {
Index: clang/lib/Format/BreakableToken.cpp
===================================================================
--- clang/lib/Format/BreakableToken.cpp
+++ clang/lib/Format/BreakableToken.cpp
@@ -41,8 +41,8 @@
static StringRef getLineCommentIndentPrefix(StringRef Comment,
const FormatStyle &Style) {
- static const char *const KnownCStylePrefixes[] = {"///<", "//!<", "///", "//",
- "//!"};
+ static const char *const KnownCStylePrefixes[] = {"///<", "//!<", "///",
+ "//", "//!", "//:"};
static const char *const KnownTextProtoPrefixes[] = {"//", "#", "##", "###",
"####"};
ArrayRef<const char *> KnownPrefixes(KnownCStylePrefixes);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93490.313545.patch
Type: text/x-patch
Size: 1507 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201223/1b07d9ec/attachment.bin>
More information about the cfe-commits
mailing list