[Minuit2] Remove GenericFunction

This class can simply be replaced by using `std::function`, and actually
it is not even used anywhere.

Author: guitargeek

math/minuit2/CMakeLists.txt

@@ -40,7 +40,6 @@ if(CMAKE_PROJECT_NAME STREQUAL ROOT)
       Minuit2/FumiliStandardMaximumLikelihoodFCN.h
       Minuit2/FunctionGradient.h
       Minuit2/FunctionMinimum.h
-      Minuit2/GenericFunction.h
       Minuit2/GradientCalculator.h
       Minuit2/HessianGradientCalculator.h
       Minuit2/InitialGradientCalculator.h

math/minuit2/inc/LinkDef.h

@@ -13,9 +13,6 @@
 #pragma link off all classes;
 #pragma link off all functions;
 
-//#pragma link C++ namespace ROOT::Minuit2;
-
-#pragma link C++ class ROOT::Minuit2::GenericFunction;
 #pragma link C++ class ROOT::Minuit2::FCNBase;
 #pragma link C++ class ROOT::Minuit2::FCNGradientBase;
 #pragma link C++ class ROOT::Minuit2::FumiliFCNBase;

math/minuit2/inc/Minuit2/FCNBase.h

@@ -12,8 +12,6 @@
 
 #include "Minuit2/MnConfig.h"
 
-#include "Minuit2/GenericFunction.h"
-
 #include <ROOT/RSpan.hxx>
 
 #include <vector>
@@ -48,10 +46,12 @@ Interface (abstract class) defining the function to be minimized, which has to b
 
  */
 
-class FCNBase : public GenericFunction {
+class FCNBase {
 
 public:
 
+   virtual ~FCNBase() = default;
+
    /**
 
       The meaning of the vector of parameters is of course defined by the user,
@@ -74,7 +74,7 @@ class FCNBase : public GenericFunction {
 
    */
 
-   double operator()(std::vector<double> const &v) const override = 0;
+   virtual double operator()(std::vector<double> const &v) const = 0;
 
    /**
 

math/minuit2/inc/Minuit2/GenericFunction.h

@@ -30,7 +30,6 @@ set(MINUIT2_HEADERS
     FumiliStandardMaximumLikelihoodFCN.h
     FunctionGradient.h
     FunctionMinimum.h
-    GenericFunction.h
     GradientCalculator.h
     HessianGradientCalculator.h
     InitialGradientCalculator.h