Skip to content
Snippets Groups Projects
Commit 65e03440 authored by Rosen Matev's avatar Rosen Matev :sunny:
Browse files

Fix tiny memory leak in HiveTestAlgorithm

parent 3a3283f6
No related branches found
No related tags found
1 merge request!1238Fix memory leak in IncidentSvc and more
......@@ -41,6 +41,6 @@ private:
Gaudi::Property<std::vector<std::string>> m_inputs{this, "Input", {}, "List of required inputs"};
Gaudi::Property<std::vector<std::string>> m_outputs{this, "Output", {}, "List of provided outputs"};
std::vector<DataObjectHandle<DataObject>*> m_inputHandles;
std::vector<DataObjectHandle<DataObject>*> m_outputHandles;
std::vector<std::unique_ptr<DataObjectHandle<DataObject>>> m_inputHandles;
std::vector<std::unique_ptr<DataObjectHandle<DataObject>>> m_outputHandles;
};
......@@ -40,7 +40,7 @@ StatusCode HiveTestAlgorithm::initialize() {
int i = 0;
for ( auto k : m_inputs ) {
debug() << "adding input key " << k << endmsg;
m_inputHandles.push_back( new DataObjectHandle<DataObject>( k, Gaudi::DataHandle::Reader, this ) );
m_inputHandles.emplace_back( std::make_unique<DataObjectHandle<DataObject>>( k, Gaudi::DataHandle::Reader, this ) );
declareProperty( "dummy_in_" + std::to_string( i ), *( m_inputHandles.back() ) );
i++;
}
......@@ -48,7 +48,8 @@ StatusCode HiveTestAlgorithm::initialize() {
i = 0;
for ( auto k : m_outputs ) {
debug() << "adding output key " << k << endmsg;
m_outputHandles.push_back( new DataObjectHandle<DataObject>( k, Gaudi::DataHandle::Writer, this ) );
m_outputHandles.emplace_back(
std::make_unique<DataObjectHandle<DataObject>>( k, Gaudi::DataHandle::Writer, this ) );
declareProperty( "dummy_out_" + std::to_string( i ), *( m_outputHandles.back() ) );
i++;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment