[Minuit2] Remove unused strategy getters and setters from MnStrategy

Settters are redundant, because if you want to create a certain
strategy, then you can just use the constructor.

Getters for the overall strategy level are unsafe, because they allow
you to introduce strategy dependence in a way that is not explicitly
clear from the public MnStrategy API.

Author: guitargeek

math/minuit2/inc/Minuit2/MnStrategy.h

@@ -46,22 +46,12 @@ class MnStrategy {
 
    int StorageLevel() const { return fStoreLevel; }
 
-   bool IsLow() const { return fStrategy == 0; }
-   bool IsMedium() const { return fStrategy == 1; }
-   bool IsHigh() const { return fStrategy == 2; }
-   bool IsVeryHigh() const { return fStrategy >= 3; }
-
    bool RefineGradientInHessian() const { return fStrategy > 0; }
 
    bool ComputeInitialHessian() const { return fStrategy == 2; }
 
    double HessianRecomputeThreshold() const;
 
-   void SetLowStrategy();
-   void SetMediumStrategy();
-   void SetHighStrategy();
-   void SetVeryHighStrategy();
-
    void SetGradientNCycles(unsigned int n) { fGradNCyc = n; }
    void SetGradientStepTolerance(double stp) { fGradTlrStp = stp; }
    void SetGradientTolerance(double toler) { fGradTlr = toler; }
@@ -88,6 +78,11 @@ class MnStrategy {
    friend class MnContours;
    MnStrategy NextLower() const;
 
+   void SetLowStrategy();
+   void SetMediumStrategy();
+   void SetHighStrategy();
+   void SetVeryHighStrategy();
+
    unsigned int fStrategy;
 
    unsigned int fGradNCyc;