[RF] Small fixup for using RooFit with new Clad 1.5

[guitargeek]

Using template functions in the generated code can cause linker errors,
which is avoided with this suggested commit.

To be backported to the 6.32 branch.

[github-actions]

Test Results

     9 files       9 suites   2d 3h 1m 9s ⏱️
 2 636 tests  2 635 ✅ 0 💤 1 ❌
22 350 runs  22 343 ✅ 0 💤 7 ❌

For more details on these failures, see this check.

Results for commit 3b3d8f7.

♻️ This comment has been updated with latest results.

[vgvassilev]

I thought we have fixed that bug in clad. Can we get a reproducer?

[guitargeek]

No, that one is not fixed, and in fact I can't reproduce it locally.

But it is very easy to circumvent: just done use std::min and std::max, like we currently do:
https://github.com/root-project/root/blob/master/roofit/roofitcore/inc/RooFit/Detail/MathFuncs.h#L547

I tried in this PR to use the std:: functions just to see what happens, but it didn't work. It's not a blocker though.

[vgvassilev]

No, that one is not fixed, and in fact I can't reproduce it locally.

But it is very easy to circumvent: just done use std::min and std::max, like we currently do: https://github.com/root-project/root/blob/master/roofit/roofitcore/inc/RooFit/Detail/MathFuncs.h#L547

I tried in this PR to use the std:: functions just to see what happens, but it didn't work. It's not a blocker though.

Do we have a bug report for it?

[guitargeek]

I have repurposed this PR to fix the current CI failures.

[vgvassilev]

Lgtm!

[vepadulano]

FYI the clad update also introduces an unused parameter warning due to

root/hist/hist/src/TFormula.cxx

Line 3161 in baf0923

Double_t *result, const Int_t result_size) {

. I don't know if you want to include the fix in this PR or I will just open a separate one

[vepadulano]

PR open at #15589