Skip to content

Conversation

@ktf
Copy link
Member

@ktf ktf commented Feb 11, 2026

No description provided.

@ktf ktf merged commit 144e229 into AliceO2Group:master Feb 11, 2026
7 checks passed
@github-actions github-actions bot changed the title Mark variables as maybe unused [ALICE3] Mark variables as maybe unused Feb 11, 2026
@ktf
Copy link
Member Author

ktf commented Feb 11, 2026

Currently broken. Merging. Someone should review why they are actually unused.

@mhemmer-cern
Copy link
Collaborator

mhemmer-cern commented Feb 11, 2026

I think theisig variable below mcDecision is also sometimes unused when removing the mcDecision (I just tried removing the unused once locally). Do they need [[maybe_unused]] as well in those cases here to not trigger variable unused or is this safe?

@ktf
Copy link
Member Author

ktf commented Feb 11, 2026

I think [[maybe_unused]] taints all derived variables, so no. In any case locally my linter does not complain, let's see.

@mhemmer-cern
Copy link
Collaborator

Okay, thank you for quickly fixing this issue.

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.

2 participants