Skip to content

Conversation

@victor-gonzalez
Copy link
Collaborator

No description provided.

@github-actions
Copy link

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

@victor-gonzalez
Copy link
Collaborator Author

@vkucera Interesting that the [cppcheck] has not found errors in the PROCESS_SWITCH macros that are above in the same file. Perhaps the fact of being associated to methods that are actually defined outside of the struct?

@vkucera
Copy link
Collaborator

vkucera commented May 21, 2025

@vkucera Interesting that the [cppcheck] has not found errors in the PROCESS_SWITCH macros that are above in the same file. Perhaps the fact of being associated to methods that are actually defined outside of the struct?

Hi @victor-gonzalez , I admit it is not clear to me yet how cppcheck detects undefined macros. Another area where it seems to give false positives are declarations of O2 filters. I still have to look into the details of the reported problems.

@victor-gonzalez victor-gonzalez enabled auto-merge (squash) May 21, 2025 21:02
Copy link
Collaborator

@alibuild alibuild left a comment

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 @victor-gonzalez.

@victor-gonzalez victor-gonzalez merged commit 37f001e into AliceO2Group:master May 22, 2025
12 of 14 checks passed
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
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