VP1BPhysSystems+VP1BanksSystems+VP1VertexSystems+VP1LegoSystems: slots -> Q_SLOTS
Rename the Qt slots 'keyword' to Q_SLOTS.
Having this in the source confuses cppcheck, which reports syntax errors. We can define slots as a macro expanding to nothing in the cppcheck configuration, but if we do that globally, there are other packages that break because they use slots as an identifier. Using Q_SLOTS instead prevents this name collision.
Merge request reports
Activity
This merge request affects 4 packages:
- graphics/VP1/VP1Systems/VP1BPhysSystems
- graphics/VP1/VP1Systems/VP1BanksSystems
- graphics/VP1/VP1Systems/VP1LegoSystems
- graphics/VP1/VP1Systems/VP1VertexSystems
Affected files list will not be printed in this case
Adding @rbianchi as watcher
added EventDisplay main review-pending-level-1 labels
✅ CI Result SUCCESS (hash 0d439fcc)Athena externals ✅ cmake ✅ make ✅ tests ✅ Full details available on this CI monitor view. Check the JIRA CI status board for known problems
✅ Athena: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 1270]added review-approved label and removed review-pending-level-1 label
mentioned in commit 71a8ea9c