[PATCH] D124708: Fix "the the" typo in documentation and user facing strings

Mark de Wever via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Apr 30 03:56:04 PDT 2022


Mordante added a comment.

Thanks for these fixes. SGTM, but I want to see the CI pass.



================
Comment at: clang/lib/StaticAnalyzer/Checkers/cert/InvalidPtrChecker.cpp:135
       FD->getNameForDiagnostic(Out, FD->getASTContext().getLangOpts(), true);
-      Out << "' call may invalidate the the result of the previous " << '\'';
+      Out << "' call may invalidate the result of the previous " << '\'';
       FD->getNameForDiagnostic(Out, FD->getASTContext().getLangOpts(), true);
----------------
I'm not familiar with the static analyzers, so I don't know whether this breaks a test.
I see the pre-commit CI couldn't apply your patch. Can you rebase this patch on main so the CI runs?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124708/new/

https://reviews.llvm.org/D124708



More information about the cfe-commits mailing list