Skip to content

Conversation

@vkucera
Copy link
Collaborator

@vkucera vkucera commented Dec 11, 2025

Please double-check the changes carefully.

@github-actions
Copy link

github-actions bot commented Dec 11, 2025

O2 linter results: ❌ 154 errors, ⚠️ 144 warnings, 🔕 91 disabled

@github-actions github-actions bot changed the title Fix problems found by Cppcheck [Common] Fix problems found by Cppcheck Dec 11, 2025
@vkucera
Copy link
Collaborator Author

vkucera commented Dec 16, 2025

@ddobrigk Which PDG code should be used to fix the duplicated condition in Common/TableProducer/selectionStudyTable.cxx?

@vkucera
Copy link
Collaborator Author

vkucera commented Dec 18, 2025

@ddobrigk Ping

ddobrigk
ddobrigk previously approved these changes Dec 18, 2025
@ddobrigk
Copy link
Collaborator

@ddobrigk Which PDG code should be used to fix the duplicated condition in Common/TableProducer/selectionStudyTable.cxx?

Sorry, overlooked this. That second loop should be removed, as it is spurious (in a part of the code that was in the end not used for anything significnat yet, so no big deal)

@vkucera
Copy link
Collaborator Author

vkucera commented Dec 19, 2025

@ddobrigk Which PDG code should be used to fix the duplicated condition in Common/TableProducer/selectionStudyTable.cxx?

Sorry, overlooked this. That second loop should be removed, as it is spurious (in a part of the code that was in the end not used for anything significnat yet, so no big deal)

I removed the duplicated block.

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