maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Improve handling of trailing block comments 

This is a follow up to r174309 to actually make it work.

llvm-svn: 174314
This commit is contained in:
Daniel Jasper 2013-02-04 08:34:57 +00:00
parent d91ae4d960
commit 14e40ec828
2 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
clang/lib/Format/Format.cpp
View File

@ -204,6 +204,11 @@ private:
tooling::Replacements Replaces;
};
static bool isTrailingComment(const AnnotatedToken &Tok) {
return Tok.is(tok::comment) &&
(Tok.Children.empty() || Tok.Children[0].MustBreakBefore);
}
class UnwrappedLineFormatter {
public:
UnwrappedLineFormatter(const FormatStyle &Style, SourceManager &SourceMgr,
@ -464,7 +469,7 @@ private:
(Previous.is(tok::l_paren) || Previous.is(tok::l_brace) ||
State.NextToken->Parent->Type == TT_TemplateOpener))
State.Stack[ParenLevel].Indent = State.Column + Spaces;
if (Previous.is(tok::comma) && Current.Type != TT_LineComment)
if (Previous.is(tok::comma) && !isTrailingComment(Current))
State.Stack[ParenLevel].HasMultiParameterLine = true;
State.Column += Spaces;
@ -690,8 +695,7 @@ private:
return true;
if (State.NextToken->Parent->is(tok::comma) &&
State.Stack.back().BreakAfterComma &&
(State.NextToken->isNot(tok::comment) ||
!State.NextToken->Children[0].MustBreakBefore))
!isTrailingComment(*State.NextToken))
return true;
if ((State.NextToken->Type == TT_CtorInitializerColon ||
(State.NextToken->Parent->ClosesTemplateDeclaration &&

5
clang/unittests/Format/FormatTest.cpp
View File

@ -1919,11 +1919,12 @@ TEST_F(FormatTest, BlockComments) {
EXPECT_EQ("someFunction(1, /* comment 1 */\n"
" 2, /* comment 2 */\n"
" 3, /* comment 3 */\n"
" aaaa);",
" aaaa,\n"
" bbbb);",
format("someFunction (1, /* comment 1 */\n"
" 2, /* comment 2 */ \n"
" 3, /* comment 3 */\n"
"aaaa );", getGoogleStyle()));
"aaaa, bbbb );", getGoogleStyle()));
}
TEST_F(FormatTest, FormatStarDependingOnContext) {