`evtContext` is copied in `ThreadLocalContext`
Currently, in https://gitlab.cern.ch/gaudi/Gaudi/blob/master/GaudiKernel/src/Lib/ThreadLocalContext.cpp#L13,
the eventcontext is copied into the thread_local
variable localContext
.
This is unfortunate if one wants to see the current state of the extension
via Gaudi::Hive::currentContext()
, since this returns the copy that was made when setCurrentContext(ctx)
is called.
However, if I was to change the thread_local variable to hold EventContext*
instead, this obv. segfaults if the eventcontext is std::move
'd.
Do we want to fix this or do we just wait until everything uses Gaudi::Algorithm
, since then we do not need a Gaudi::Hive::currentContext()
in the execute
?
@clemenci fyi