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
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to