TrigT1CaloByteStream: Make CpByteStream* tools const.

Make remaining CpByteStream* tools const.

This merge request affects 1 package:

• Trigger/TrigT1/TrigT1CaloByteStream

Adding @kwhalen ,@afaulkne as watchers

added Trigger master review-pending-level-1 labels

CI Result FAILURE (hash cd21b6aa)

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

Please address the failed AthSimulation test with "CI_TEST/CI_TEST_RUNNER: TEST FAILURE: SimulationTier0Test_required returned code 255". (L1)

added review-user-action-required label and removed review-pending-level-1 label

Jenkins please retry a build

This merge request affects 1 package:

• Trigger/TrigT1/TrigT1CaloByteStream

Adding @kwhalen ,@afaulkne as watchers

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash cd21b6aa)

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

A whole bunch of changes to make a Cp tool methods const. Quite a bit of code internally had to be changed, and extra parameters to haul around the tool's internal state had to be added in various places.

Code changes look ok, but would prefer someone else who knows more about trigger take a quick look.

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

added review-user-action-required label and removed review-pending-level-2 label

added 1 commit

• c772c869 - address comments

Compare with previous version

resolved all threads

added review-pending-level-1 label

removed review-user-action-required label

This merge request affects 1 package:

• Trigger/TrigT1/TrigT1CaloByteStream

Adding @kwhalen ,@afaulkne as watchers

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

CI Result FAILURE (hash c772c869)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

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

Failure is unrelated.

merged

mentioned in commit 878a0c15

added sweep:ignore label