VP1TrackSystems: 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.

This merge request affects 1 package:

• graphics/VP1/VP1Systems/VP1TrackSystems

This merge request affects 8 files:

• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/AscObjSelectionManager.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/TrackCollHandleBase.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/TrackCollHandle_SimulationTracks.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/TrackCollHandle_TruthTracks.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/TrackCollectionSettingsButton.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/TrackLODManager.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/TrackSystemController.h
• graphics/VP1/VP1Systems/VP1TrackSystems/VP1TrackSystems/VP1TrackSystem.h

Adding @jchapman ,@rbianchi as watchers

added EventDisplay main review-pending-level-1 labels

CI Result SUCCESS (hash 697e5633)

	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 1269]

Looks good from L1 perspective.

Cheers, Anne

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 918464ac