Introduced DeprecatedDynamicDataObjectHandle
This transitional class will help moving all code to Handles even if some usage requires 'dynamic' behavior, i.e. to declare handles at run time only and not registering them with their owner.
Again should no be used for long term code, only for helping in the transition. It is bound to be marked Deprecated and then dropped as the name explicitely says.
Note that it needed a new constructor in the DataObjectHandleBase but that one is directly marked deprecated and used only in the new class where the deprecation warning is ignored
Merge request reports
Activity
- Resolved by Sebastien Ponce
@graven following our discussions.
This is more complicated than expected - as expected, no ? ;-) Basically I could not make it work without the use of a const_cast... Maybe it's ok for a class called DeprecatedDynamicDataObjectHandle - yes long, but at least disgusting enough that people won't use it without a good reason. The problem is that the base class DataHandle insists on having an non const owner. And of course in Gaudi::algorithm, the execute method is const so I cannot provide that.
mentioned in merge request lhcb/LHCb!3001 (merged)
- [2021-04-15 17:29] Validation started with lhcb-master-mr#2208
- [2021-04-16 09:19] Validation started with lhcb-master-mr#2211
- [2021-06-09 15:46] Validation started with lhcb-gaudi-head#2953
- [2021-06-10 00:05] Validation started with lhcb-gaudi-head#2954
- [2021-06-10 00:56] Validation started with lhcb-run2-gaudi-head#144
Edited by Software for LHCbadded 1 commit
- 51801802 - Introduced DeprecatedDynamicDataObjectHandle
added 1 commit
- f7a953f3 - Introduced DeprecatedDynamicDataObjectHandle
- Resolved by Sebastien Ponce
added 1 commit
- ed343ec0 - Introduced DeprecatedDynamicDataObjectHandle
added 1 commit
- f1878aff - Introduced DeprecatedDynamicDataObjectHandle
assigned to @clemenci
changed milestone to %v36r0
added lhcb-gaudi-head label
mentioned in commit 80edbf9b