Various minor clean-ups in the HLT domain

A little Christmas clean-up. I went through several (but not all) packages in the HLT directory and made minor fixes, including:

fixing typos

removing empty constructor / destructor

flake8 fixes

CMake fixes

removing obsolete files

adding copyright statement

@fwinkl, related question - is the TrigPython package needed? It consists only of a dictionary with one item, nothing else. How is this used?

This merge request affects 5 packages:

• HLT/Trigger/TrigControl/TrigCommon
• HLT/Trigger/TrigControl/TrigExamples/TrigExPartialEB
• HLT/Trigger/TrigControl/TrigKernel
• HLT/Trigger/TrigControl/TrigPSC
• HLT/Trigger/TrigControl/TrigServices

Adding @wiedenma ,@bernius ,@fwinkl ,@rbielski as watchers

added Trigger master review-pending-level-1 labels

Nice work! I am missing some though The dictionary was needed in 21.1 here https://gitlab.cern.ch/atlas/athena/blob/21.1/HLT/Trigger/TrigControl/TrigPSC/share/TrigPSCPythonDbSetup.py#L63 to be able to override OutputLevels when running from the DB. I haven't really tested/implemented this feature in master yet. It may not actually be needed. Let me take a look at that and I make a separate MR if needed.

added 1 commit

• f576c2dd - Various minor clean-ups in the HLT directory

Compare with previous version

resolved all threads

This merge request affects 5 packages:

• HLT/Trigger/TrigControl/TrigCommon
• HLT/Trigger/TrigControl/TrigExamples/TrigExPartialEB
• HLT/Trigger/TrigControl/TrigKernel
• HLT/Trigger/TrigControl/TrigPSC
• HLT/Trigger/TrigControl/TrigServices

Adding @wiedenma ,@bernius ,@fwinkl ,@rbielski as watchers

CI Result SUCCESS (hash 06865b97)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 7369]

CI Result SUCCESS (hash f576c2dd)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 7378]

The changes look ok. Adding review-approved label.

L1 shifter

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

merged

mentioned in commit e5f6516c

mentioned in merge request !29089 (merged)

added sweep:ignore label