github-actions[bot] wrote:
<!--LLVM CODE LINT COMMENT: {clang-tidy}-->
:warning: C/C++ code linter clang-tidy found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git diff -U0 origin/main..HEAD --
clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp |
python3 clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py \
-path build -p1
``````````
</details>
<details>
<summary>
View the diff from clang-tidy here.
</summary>
``````````diff
Running clang-tidy in 1 threads...
/__w/llvm-project/llvm-project/clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp:22:19:
warning: invalid case style for variable 'beforeArgumentsRange'
[readability-identifier-naming]
22 | CharSourceRange beforeArgumentsRange = Lexer::makeFileCharRange(
| ^~~~~~~~~~~~~~~~~~~~
| BeforeArgumentsRange
23 | CharSourceRange::getCharRange(Call->getBeginLoc(),
Arg->getBeginLoc()),
24 | SM, LangOpts);
25 | CharSourceRange AfterArgumentsRange = Lexer::makeFileCharRange(
26 | CharSourceRange::getCharRange(Call->getEndLoc(),
27 |
Call->getEndLoc().getLocWithOffset(1)),
28 | SM, LangOpts);
29 |
30 | if (beforeArgumentsRange.isValid() && AfterArgumentsRange.isValid()) {
| ~~~~~~~~~~~~~~~~~~~~
| BeforeArgumentsRange
31 | Diag << FixItHint::CreateRemoval(beforeArgumentsRange)
| ~~~~~~~~~~~~~~~~~~~~
| BeforeArgumentsRange
``````````
</details>
https://github.com/llvm/llvm-project/pull/154223
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits