diff --git a/lib/Format/Format.cpp b/lib/Format/Format.cpp index c08bcf4f5c109a66619c2aac555af846731188ef..60b2f56745f9c0898d14b6fa786835c7c1e5cba8 100644 --- a/lib/Format/Format.cpp +++ b/lib/Format/Format.cpp @@ -822,19 +822,23 @@ private: TokenAnnotation::TokenType determineStarAmpUsage(unsigned Index, bool IsRHS) { if (Index == Annotations.size()) return TokenAnnotation::TT_Unknown; + const FormatToken &PrevToken = Line.Tokens[Index - 1]; + const FormatToken &NextToken = Line.Tokens[Index + 1]; - if (Index == 0 || Line.Tokens[Index - 1].Tok.is(tok::l_paren) || - Line.Tokens[Index - 1].Tok.is(tok::comma) || - Line.Tokens[Index - 1].Tok.is(tok::kw_return) || - Line.Tokens[Index - 1].Tok.is(tok::colon) || + if (Index == 0 || PrevToken.Tok.is(tok::l_paren) || + PrevToken.Tok.is(tok::comma) || PrevToken.Tok.is(tok::kw_return) || + PrevToken.Tok.is(tok::colon) || Annotations[Index - 1].Type == TokenAnnotation::TT_BinaryOperator) return TokenAnnotation::TT_UnaryOperator; - if (Line.Tokens[Index - 1].Tok.isLiteral() || - Line.Tokens[Index + 1].Tok.isLiteral() || - Line.Tokens[Index + 1].Tok.is(tok::kw_sizeof)) + if (PrevToken.Tok.isLiteral() || NextToken.Tok.isLiteral() || + NextToken.Tok.is(tok::kw_sizeof)) return TokenAnnotation::TT_BinaryOperator; + if (NextToken.Tok.is(tok::comma) || NextToken.Tok.is(tok::r_paren) || + NextToken.Tok.is(tok::greater)) + return TokenAnnotation::TT_PointerOrReference; + // It is very unlikely that we are going to find a pointer or reference type // definition on the RHS of an assignment. if (IsRHS) diff --git a/unittests/Format/FormatTest.cpp b/unittests/Format/FormatTest.cpp index 22da93ebe3c1907b90ea822c20f1f191e6a625af..b52a9fd7c372e698c96be440595a08a1a35a80e4 100644 --- a/unittests/Format/FormatTest.cpp +++ b/unittests/Format/FormatTest.cpp @@ -699,6 +699,9 @@ TEST_F(FormatTest, UnderstandsUsesOfStarAndAmp) { verifyFormat("A<int **> a;"); verifyFormat("A<int *, int *> a;"); verifyFormat("A<int **, int **> a;"); + verifyFormat("Type *A = static_cast<Type *>(P);"); + verifyFormat("Type *A = (Type *) P;"); + verifyFormat("Type *A = (vector<Type *, int *>) P;"); verifyGoogleFormat("int main(int argc, char** argv) {\n}"); verifyGoogleFormat("A<int*> a;");