diff --git a/clang/lib/Format/BreakableToken.cpp b/clang/lib/Format/BreakableToken.cpp --- a/clang/lib/Format/BreakableToken.cpp +++ b/clang/lib/Format/BreakableToken.cpp @@ -779,11 +779,14 @@ const char FirstCommentChar = Lines[i][IndentPrefix.size()]; const unsigned FirstCharByteSize = encoding::getCodePointNumBytes(FirstCommentChar, Encoding); - return encoding::columnWidth( - Lines[i].substr(IndentPrefix.size(), FirstCharByteSize), - Encoding) == 1 && - (FirstCommentChar == '\\' || isPunctuation(FirstCommentChar) || - isHorizontalWhitespace(FirstCommentChar)); + if (encoding::columnWidth( + Lines[i].substr(IndentPrefix.size(), FirstCharByteSize), + Encoding) != 1) + return false; + if (FirstCommentChar == '#') + return false; + return FirstCommentChar == '\\' || isPunctuation(FirstCommentChar) || + isHorizontalWhitespace(FirstCommentChar); }; // On the first line of the comment section we calculate how many spaces diff --git a/clang/unittests/Format/FormatTestComments.cpp b/clang/unittests/Format/FormatTestComments.cpp --- a/clang/unittests/Format/FormatTestComments.cpp +++ b/clang/unittests/Format/FormatTestComments.cpp @@ -91,6 +91,9 @@ "// line 2\n" "void f() {}\n"); + EXPECT_EQ("// comment\n", format("//comment\n")); + EXPECT_EQ("// #comment\n", format("//#comment\n")); + EXPECT_EQ("// comment\n" "// clang-format on\n", format("//comment\n" @@ -3302,6 +3305,7 @@ format(NoTextInComment, Style)); Style.SpacesInLineCommentPrefix.Minimum = 0; + EXPECT_EQ("//#comment", format("//#comment", Style)); EXPECT_EQ("//\n" "\n" "void foo() { //\n" @@ -3310,6 +3314,7 @@ format(NoTextInComment, Style)); Style.SpacesInLineCommentPrefix.Minimum = 5; + EXPECT_EQ("// #comment", format("//#comment", Style)); EXPECT_EQ("//\n" "\n" "void foo() { //\n" @@ -3463,6 +3468,7 @@ format(Code, Style)); Style.SpacesInLineCommentPrefix = {0, 0}; + EXPECT_EQ("//#comment", format("// #comment", Style)); EXPECT_EQ("//Free comment without space\n" "\n" "//Free comment with 3 spaces\n"