Skip to content

VP1MissingEtSystems+VP1SimHitSystems+VP1MCSystems: slots -> Q_SLOTS

Scott Snyder requested to merge ssnyder/athena:slots.VP1MissingEtSystems-20231120 into main

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