Clean out constructors of BCM_RawDataProviderTool & co

As follow-up to !60451 (merged), simplify and modernize the BCM ByteStream conversion tools.

This merge request affects 3 packages:

• InnerDetector/InDetConfig
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv
• InnerDetector/InDetExample/InDetRecExample

Affected files list will not be printed in this case

Adding @goetz ,@sroe as watchers

added InnerDetector master review-pending-level-1 labels

CI Result SUCCESS (hash b0f5458e)

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

Looks good. Karolos (L2)

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

added review-approved-point1 label

added review-approved-tier0 label

hi @wlampl, looks like we have merge conflicts here after merging !60451 (merged)

added review-user-action-required label

added 247 commits

• b0f5458e...f008db53 - 246 commits from branch atlas:master
• f8a3482b - Merge branch 'master' into 'BcmRawDataProviderNoConstr'

Compare with previous version

This merge request affects 1 package:

• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv

This merge request affects 6 files:

• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/BCM_RawDataByteStreamCnv/BCM_RawDataProvider.h
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/BCM_RawDataByteStreamCnv/BCM_RawDataProviderTool.h
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/BCM_RawDataByteStreamCnv/BCM_RodDecoder.h
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/src/BCM_RawDataProvider.cxx
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/src/BCM_RawDataProviderTool.cxx
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/src/BCM_RodDecoder.cxx

Adding @goetz ,@sroe as watchers

added review-pending-level-1 label and removed review-approved review-approved-point1 review-approved-tier0 review-user-action-required labels

CI Result SUCCESS (hash f8a3482b)

	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-CC7 66056]

Just one minor comment. Otherwise the changes look good.

Jason (L1)

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

resolved all threads

added 1 commit

• b3f431ed - FIx spelling in comment

Compare with previous version

Build area was cleaned as per request posted in the DB. The full software build will be performed

This merge request affects 1 package:

• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv

This merge request affects 6 files:

• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/BCM_RawDataByteStreamCnv/BCM_RawDataProvider.h
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/BCM_RawDataByteStreamCnv/BCM_RawDataProviderTool.h
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/BCM_RawDataByteStreamCnv/BCM_RodDecoder.h
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/src/BCM_RawDataProvider.cxx
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/src/BCM_RawDataProviderTool.cxx
• InnerDetector/InDetEventCnv/BCM_RawDataByteStreamCnv/src/BCM_RodDecoder.cxx

Adding @goetz ,@sroe as watchers