]> granicus.if.org Git - clang/commitdiff
[analyzer] Correct the summary violation diagnostics for the retain count checker
authorGeorge Karpenkov <ekarpenkov@apple.com>
Fri, 21 Dec 2018 19:13:28 +0000 (19:13 +0000)
committerGeorge Karpenkov <ekarpenkov@apple.com>
Fri, 21 Dec 2018 19:13:28 +0000 (19:13 +0000)
It should be in the past tense.

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

lib/StaticAnalyzer/Checkers/RetainCountChecker/RetainCountDiagnostics.cpp
test/Analysis/osobject-retain-release.cpp

index 8fdd105f189562b3aade854a8fb407d867648a6a..74dd1e149ef3e66d7b8c779822a329efbdab9345 100644 (file)
@@ -285,7 +285,7 @@ annotateConsumedSummaryMismatch(const ExplodedNode *N,
         os << "Parameter '";
         PVD->getNameForDiagnostic(os, PVD->getASTContext().getPrintingPolicy(),
                                   /*Qualified=*/false);
-        os << "' is marked as consuming, but the function does not consume "
+        os << "' is marked as consuming, but the function did not consume "
            << "the reference\n";
       }
     }
index 2efd20709bb59c808e10b577e7fd695eb64b5b5f..4c23cdc6e08a4d89de5007e248af9ec02213b4c2 100644 (file)
@@ -99,7 +99,7 @@ bool os_consume_violation_two_args(OS_CONSUME OSObject *obj, bool extra) {
     escape(obj);
     return true;
   }
-  return false; // expected-note{{Parameter 'obj' is marked as consuming, but the function does not consume the reference}}
+  return false; // expected-note{{Parameter 'obj' is marked as consuming, but the function did not consume the reference}}
 }
 
 bool os_consume_violation(OS_CONSUME OSObject *obj) {
@@ -108,7 +108,7 @@ bool os_consume_violation(OS_CONSUME OSObject *obj) {
     escape(obj);
     return true;
   }
-  return false; // expected-note{{Parameter 'obj' is marked as consuming, but the function does not consume the reference}}
+  return false; // expected-note{{Parameter 'obj' is marked as consuming, but the function did not consume the reference}}
 }
 
 void os_consume_ok(OS_CONSUME OSObject *obj) {