Skip to content

IsolationSelection rm un-needed const leading to const_cast,

Christos Anastopoulos requested to merge ATLAS-EGamma/athena:Isolation_selection_const_mess into master

IsolationSelection rm un-needed const leading to const_cast, update test to work with shallow copies.

This follow a discussions in !57915 (merged)

Assuming the indent is really to work with Shallow copy container we do not need to pass them const and const_cast. This allowed me to remove one of two const_cast. Ping @fwinkl . Also lets update the test to do this .

There is another one which I assume is also prb due to a confusing part of the interface (we modify copies or not etc), but as this is different than the one I commented leaving this to @goblirsc and @schaffer

Merge request reports