Skip to content

[Common] Fix issue 14173#16814

Merged
dsekihat merged 4 commits into
AliceO2Group:masterfrom
amaringarcia:master
Jun 30, 2026
Merged

[Common] Fix issue 14173#16814
dsekihat merged 4 commits into
AliceO2Group:masterfrom
amaringarcia:master

Conversation

@amaringarcia

@amaringarcia amaringarcia commented Jun 25, 2026 •
edited by vkucera
Loading

Copy link
Copy Markdown
Contributor

Fixes #14173

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Fixing issues/14173 [Common] Fixing issues/14173 Jun 25, 2026
@alibuild alibuild mentioned this pull request Jun 25, 2026
Closed
@vkucera

vkucera commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Thanks a lot @amaringarcia !

@vkucera

vkucera commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

It seems the identicalConditionAfterEarlyExit warnings are still there.

vkucera
vkucera reviewed Jun 25, 2026
View reviewed changes
Comment thread Common/TableProducer/occupancyTableProducer.cxx Outdated
@vkucera vkucera changed the title [Common] Fixing issues/14173 [Common] Fix issue 14173 Jun 25, 2026
@amaringarcia

amaringarcia commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Good morning,
now all the warnings/issues are solved (I think).
Can somebody approve and merge this pull request?
Thanks in advance.
Best regards,
Ana

@amaringarcia

amaringarcia commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

@dsekihat could you approve this pull request? also allow for the merging?

@vkucera

vkucera commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

@amaringarcia Can you please check whether these can be fixed as well?

Common/TableProducer/occupancyTableProducer.cxx:480:142: warning: Identical condition 'meanRobustTableMode==checkTableMode', second condition is always false [identicalConditionAfterEarlyExit]
Common/TableProducer/occupancyTableProducer.cxx:1806:102: warning: Identical condition 'qaMode==checkQAMode', second condition is always false [identicalConditionAfterEarlyExit]

@alibuild

alibuild commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 685aa0c at 2026-06-29 07:24:

## sw/BUILD/O2Physics-latest/log
c++: fatal error: Killed signal terminated program cc1plus
c++: fatal error: Killed signal terminated program cc1plus
ninja: build stopped: subcommand failed.

Full log here.

@amaringarcia

amaringarcia commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

@ddobrigk @dsekihat @iarsene could some of you approve and merge this pull request?
Thanks in advance

@dsekihat dsekihat merged commit 990339d into AliceO2Group:master Jun 30, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vkucera vkucera vkucera left review comments
@dsekihat dsekihat dsekihat approved these changes
@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner
@jgrosseo jgrosseo Awaiting requested review from jgrosseo jgrosseo is a code owner
@iarsene iarsene Awaiting requested review from iarsene iarsene is a code owner
@ktf ktf Awaiting requested review from ktf ktf is a code owner
@ddobrigk ddobrigk Awaiting requested review from ddobrigk ddobrigk is a code owner

Assignees

No one assigned

Labels

common

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Potential bugs in occupancyTableProducer.cxx

4 participants

@amaringarcia @vkucera @alibuild @dsekihat