]> granicus.if.org Git - clang/commitdiff
Fix formatting of overloaded assignment operators.
authorDaniel Jasper <djasper@google.com>
Thu, 11 Apr 2013 08:48:20 +0000 (08:48 +0000)
committerDaniel Jasper <djasper@google.com>
Thu, 11 Apr 2013 08:48:20 +0000 (08:48 +0000)
Before: SomeType &operator=(const SomeType & S);
After:  SomeType &operator=(const SomeType &S);

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@179270 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Format/TokenAnnotator.cpp
unittests/Format/FormatTest.cpp

index 42cbf04aec9296062abb8116b2335f1b0d814beb..08d9b10694a0af15d57327241ca6fa187633c4a6 100644 (file)
@@ -563,7 +563,8 @@ private:
   };
 
   void determineTokenType(AnnotatedToken &Current) {
-    if (getPrecedence(Current) == prec::Assignment) {
+    if (getPrecedence(Current) == prec::Assignment &&
+        (!Current.Parent || Current.Parent->isNot(tok::kw_operator))) {
       Contexts.back().IsExpression = true;
       for (AnnotatedToken *Previous = Current.Parent;
            Previous && Previous->isNot(tok::comma);
index 5b5f483c62b84bc7804b740342f5fd390496ed95..09fbce830e37b3591f91d4ae51e9fafb8c5bf43f 100644 (file)
@@ -2374,6 +2374,7 @@ TEST_F(FormatTest, UndestandsOverloadedOperators) {
                "               const aaaaaaaaaaaaaaaaaaaaa &right) {\n"
                "  return left.group < right.group;\n"
                "}");
+  verifyFormat("SomeType &operator=(const SomeType &S);");
 
   verifyGoogleFormat("operator void*();");
   verifyGoogleFormat("operator SomeType<SomeType<int>>();");