[Tutorial,PWGCF,PWGEM,PWGLF,PWGUD] Use const& and std::move to avoid copies
#33746
vkuceraAnnotations
10 errors, 1 warning, and 2 notices
|
[magic-number]:
Tutorials/PWGLF/Resonance/resonancesCombine.cxx#L192
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
Tutorials/PWGLF/Resonance/resonancesCombine.cxx#L190
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
Tutorials/PWGLF/Resonance/resonancesCombine.cxx#L162
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[root/lorentz-vector]:
Tutorials/PWGLF/Resonance/resonancesCombine.cxx#L125
Do not use the TLorentzVector legacy class. Use std::array with RecoDecay methods or the ROOT::Math::LorentzVector template instead.
|
|
[root/lorentz-vector]:
Tutorials/PWGLF/Resonance/resonancesCombine.cxx#L32
Do not use the TLorentzVector legacy class. Use std::array with RecoDecay methods or the ROOT::Math::LorentzVector template instead.
|
|
[name/workflow-file]:
Tutorials/PWGCF/EventPlane/src/qVectorstutorial.cxx#L1
Name of a workflow file must match the name of the main struct in it (without the PWG prefix). (Class implementation files should be in "Core" directories.)
|
|
[magic-number]:
Tutorials/PWGCF/EventPlane/src/qVectorstutorial.cxx#L145
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
Tutorials/PWGCF/EventPlane/src/qVectorstutorial.cxx#L138
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[magic-number]:
Tutorials/PWGCF/EventPlane/src/qVectorstutorial.cxx#L128
Avoid magic numbers in expressions. Assign the value to a clearly named variable or constant.
|
|
[const-ref-in-for-loop]:
Tutorials/PWGCF/EventPlane/src/qVectorstutorial.cxx#L141
Use constant references for non-modified iterators in range-based for loops.
|
|
O2 linter
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: thollander/actions-comment-pull-request@v3. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
O2 linter
To tolerate certain issues in a directory, add a line with the test name in "o2linter_config".
|
|
O2 linter
Exceptionally, you can disable a test for a line by adding a comment with "o2-linter: disable=" followed by the name of the test and parentheses with a reason for the exception.
|