Merge pull request cms-sw#25450 from Dr15Jones/clangMuonAnalysisMomen…

…tumScaleCalibrartion fix clang warnings in MuonAnalysis/MomentumScaleCalibration
forthommel · Dec 10, 2018 · c1000bc · c1000bc
2 parents 95c317f + 5123d91
commit c1000bc
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/MuonAnalysis/MomentumScaleCalibration/plugins/Histograms.h b/MuonAnalysis/MomentumScaleCalibration/plugins/Histograms.h
@@ -1423,7 +1423,7 @@ class HLikelihoodVSPart : public Histograms
     Fill(CLHEP::HepLorentzVector(p4.x(),p4.y(),p4.z(),p4.t()), likeValue);
   }
 
-   virtual void Fill(CLHEP::HepLorentzVector momentum, double likeValue) { 
+  void Fill(const CLHEP::HepLorentzVector& momentum, const double& likeValue) override { 
      hLikeVSPt_->Fill(momentum.perp(),likeValue); 
      hLikeVSEta_->Fill(momentum.eta(),likeValue); 
      hLikeVSPhi_->Fill(momentum.phi(),likeValue); 
@@ -1903,6 +1903,7 @@ class HCovarianceVSxy : public Histograms
     }
   }
 
+  using Histograms::Get;
   double Get( const double & x, const double & y ) const {
     // Need to convert the (x,y) values to the array indeces
     int xIndex = getXindex(x);