Skip to content

NSWCalibTool - Replace suspicious TF1 by the std::function equivalent

Johannes Junggeburth requested to merge jojungge/athena:ReplaceLorentzTF1_ByStdFunc into 22.0

Hi everybody,

  • Although TF1::Eval claims to be a const method, it's always worth to have a closer look into its guts . As you can see, it calls a non-const function which is super suspicious to me. Given that the NSWCalibTool just uses it for a polynomial evaluation, it's better to replace it by a std::function.

ATLASRECTS-7101/ATLASRECTS-7107/ATLASRECTS-7149

Tagging: @pscholer, @chchau

Edited by Johannes Junggeburth

Merge request reports