Merge branch 'GAUDI-1183' into 'master'
reorganize DataHandleHolder visitor acces to avoid non-retrieved tools This is not a perfect fix, more a workaround. The method `initToolHandles` have been modified to fill the list of used `ToolHandle`s only with the tools retrieved explicitly in the `initialize` method. `GaudiCommon` required the introduction of a hook so that the custom _fixer_ visitor could be created in the `GaudiCommon::initialize`, but used only at the end of `PBASE::sysInitialize` (eb392741). I also fixed an issue (double delete) with the use of `ToolHandle` from `GaudiCommon` (0a1bec8). Fixes GAUDI-1183. See merge request !120
Showing
- GaudiAlg/GaudiAlg/GaudiCommon.h 1 addition, 1 deletionGaudiAlg/GaudiAlg/GaudiCommon.h
- GaudiAlg/GaudiAlg/GaudiCommonImp.h 2 additions, 0 deletionsGaudiAlg/GaudiAlg/GaudiCommonImp.h
- GaudiAlg/src/lib/GaudiCommon.icpp 10 additions, 12 deletionsGaudiAlg/src/lib/GaudiCommon.icpp
- GaudiExamples/src/AlgTools/MyGaudiAlgorithm.cpp 2 additions, 0 deletionsGaudiExamples/src/AlgTools/MyGaudiAlgorithm.cpp
- GaudiExamples/src/AlgTools/MyGaudiAlgorithm.h 2 additions, 0 deletionsGaudiExamples/src/AlgTools/MyGaudiAlgorithm.h
- GaudiExamples/tests/qmtest/refs/AlgTools2.ref 28 additions, 35 deletionsGaudiExamples/tests/qmtest/refs/AlgTools2.ref
- GaudiExamples/tests/qmtest/refs/AlgTools_pyopts.ref 18 additions, 18 deletionsGaudiExamples/tests/qmtest/refs/AlgTools_pyopts.ref
- GaudiKernel/GaudiKernel/AlgTool.h 3 additions, 0 deletionsGaudiKernel/GaudiKernel/AlgTool.h
- GaudiKernel/GaudiKernel/Algorithm.h 24 additions, 61 deletionsGaudiKernel/GaudiKernel/Algorithm.h
- GaudiKernel/GaudiKernel/IDataHandleHolder.h 2 additions, 3 deletionsGaudiKernel/GaudiKernel/IDataHandleHolder.h
- GaudiKernel/GaudiKernel/ToolHandle.h 11 additions, 1 deletionGaudiKernel/GaudiKernel/ToolHandle.h
- GaudiKernel/src/Lib/AlgTool.cpp 20 additions, 19 deletionsGaudiKernel/src/Lib/AlgTool.cpp
- GaudiKernel/src/Lib/Algorithm.cpp 18 additions, 23 deletionsGaudiKernel/src/Lib/Algorithm.cpp
Loading
Please register or sign in to comment