Skip to content

[PWGLF] Fix secondary Lambda histogram name in derivedlambdakzeroanalysis.cxx - #17479

Merged
romainschotter merged 1 commit into
masterfrom
romainschotter-patch-2
Aug 18, 2026
Merged

[PWGLF] Fix secondary Lambda histogram name in derivedlambdakzeroanalysis.cxx#17479
romainschotter merged 1 commit into
masterfrom
romainschotter-patch-2

Conversation

@romainschotter

Copy link
Copy Markdown
Collaborator
  • Fix secondary Lambda histogram name in derivedlambdakzeroanalysis.cxx

@github-actions github-actions Bot added the pwglf label Aug 17, 2026
@github-actions github-actions Bot changed the title Fix secondary Lambda histogram name in derivedlambdakzeroanalysis.cxx [PWGLF] Fix secondary Lambda histogram name in derivedlambdakzeroanalysis.cxx Aug 17, 2026
@github-actions

Copy link
Copy Markdown

O2 linter results: ❌ 40 errors, ⚠️ 1 warnings, 🔕 0 disabled

@romainschotter
romainschotter marked this pull request as ready for review August 17, 2026 20:03
@romainschotter
romainschotter enabled auto-merge (squash) August 17, 2026 20:03

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @romainschotter.

@alibuild

alibuild commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 19c10d2 at 2026-08-18 03:45:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 17 errors and 143 warnings.'
Found 17 errors and 143 warnings.
++ [[ 17 -gt 0 ]]
++ cat /sw/BUILD/0e46a1af452f0f9c7628be223cebbc6480a855a0/O2Physics-code-check/errors.txt
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:90:9: error: function-like macro 'BITSET' used; consider a 'constexpr' template function [cppcoreguidelines-macro-usage,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:91:9: error: function-like macro 'BITCHECK' used; consider a 'constexpr' template function [cppcoreguidelines-macro-usage,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:102:8: error: constructor does not initialize these fields: isRun3, ccdb, mRunNumber, magField, maskTopological, maskTopoNoV0Radius, maskTopoNoDCANegToPV, maskTopoNoDCAPosToPV, maskTopoNoCosPA, maskTopoNoDCAV0Dau, maskTrackProperties, maskK0ShortSpecific, maskLambdaSpecific, maskAntiLambdaSpecific, maskSelectionK0Short, maskSelectionLambda, maskSelectionAntiLambda, secondaryMaskSelectionLambda, secondaryMaskSelectionAntiLambda [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:183:20: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:201:67: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:466:16: error: redundant boolean literal in conditional assignment [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1273:5: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1300:28: error: the parameter 'collision' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1361:44: error: the parameter 'v0' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1361:59: error: the parameter 'collision' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1624:37: error: the parameter 'v0' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1807:29: error: the parameter 'v0' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:2210:46: error: the parameter #1 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:2233:31: error: the parameter 'v0' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:2287:35: error: the parameter 'collision' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:2734:5: error: redundant return statement at the end of a function with a void return type [readability-redundant-control-flow,-warnings-as-errors]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:2798:5: error: redundant return statement at the end of a function with a void return type [readability-redundant-control-flow,-warnings-as-errors]
++ [[ 143 -gt 0 ]]
++ cat /sw/BUILD/0e46a1af452f0f9c7628be223cebbc6480a855a0/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1104:24: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1106:23: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1108:27: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1271:42: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1273:47: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1275:55: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1277:46: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1279:50: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1281:47: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1313:54: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1338:22: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1340:18: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1342:17: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1353:46: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1355:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1366:47: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1368:50: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1370:61: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1372:61: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1374:45: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1376:53: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1381:54: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Strangeness/derivedlambdakzeroanalysis.cxx:1384:55: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@vkucera

vkucera commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

@romainschotter Please create PRs from your fork repository to avoid polluting the upstream repository with references to personal branches.

@romainschotter
romainschotter merged commit 7be3b74 into master Aug 18, 2026
12 of 16 checks passed
@romainschotter
romainschotter deleted the romainschotter-patch-2 branch August 18, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants