Properly handle object lifetime

[ktf]

Taking ownership of DPL provided objects is wrong.

[Barthelemy]

I'll let @knopers8 review it
Then I will make a new minor version.

[knopers8]

Out of curiosity, did it cause trouble or are you just fixing a warning? It looks like the pointer is never accessed outside of this scope.

And speaking of stealing the framework's belongings, we also do this ;)

QualityControl/Framework/src/TaskRunner.cxx

Line 95 in 3eec58e

iCtx.services().get<CallbackService>().set<CallbackService::Id::Start>([this, services = iCtx.services()]() mutable { start(services); });