[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