Resolve inconsistency in L1Threshold descriptive comments

Corrects a comment which contradicts the one where the m_priority member variable is defined, as well as the implementation in L1ThresholdBase.icc and what has been assumed in the menu.

Tagging @stelzer to make sure I didn't miss something.

This merge request affects 1 package:

• Trigger/TrigConfiguration/TrigConfData

This merge request affects 1 file:

• Trigger/TrigConfiguration/TrigConfData/TrigConfData/L1ThresholdBase.h

Adding @tamartin ,@jmharris ,@asonay ,@oliveirg ,@orlando ,@paulama ,@cmorenom ,@mark as watchers

added 23.0 Trigger analysis-review-required review-pending-level-1 labels

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)(FullSim|Hit).*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash 5f9dc770)

	Athena	AnalysisBase	AthAnalysis	DetCommon
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
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 73174]

lgtm

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

I found another similar comment, let's hold this until I fix that too

resolved all threads

added analysis-review-approved label and removed analysis-review-required label

added review-approved-point1 label

mentioned in commit 5d0907c9

merged

mentioned in merge request !63972 (merged)