Skip to content
Snippets Groups Projects

VP1BPhysSystems+VP1BanksSystems+VP1VertexSystems+VP1LegoSystems: slots -> Q_SLOTS

Merged Scott Snyder requested to merge ssnyder/athena:slots.VP1BPhysSystems-20231116 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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading