From 7e7c165757fdcb989b109e6afe5f90ae63e26e5a Mon Sep 17 00:00:00 2001
From: Patrick Gartung <gartung@fnal.gov>
Date: Wed, 24 May 2017 16:41:36 -0500
Subject: [PATCH] fix gcc700 warning: class  has virtual functions and
 accessible non-virtual destructor [-Wnon-virtual-dtor]

---
 SimCalorimetry/CaloSimAlgos/interface/CaloShapes.h             | 2 +-
 SimCalorimetry/CaloSimAlgos/interface/CaloVHitCorrection.h     | 1 +
 SimCalorimetry/CaloSimAlgos/interface/CaloVHitFilter.h         | 1 +
 SimCalorimetry/CaloSimAlgos/interface/CaloVSimParameterMap.h   | 1 +
 SimCalorimetry/HGCalSimProducers/interface/HGCDigitizerBase.h  | 3 +--
 .../HcalZeroSuppressionProducers/src/HcalZSAlgoEnergy.h        | 1 +
 .../HcalZeroSuppressionProducers/src/HcalZSAlgoRealistic.h     | 2 +-
 .../HcalZeroSuppressionProducers/src/HcalZeroSuppressionAlgo.h | 1 +
 8 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/SimCalorimetry/CaloSimAlgos/interface/CaloShapes.h b/SimCalorimetry/CaloSimAlgos/interface/CaloShapes.h
index b63402645c922..24463466f57d0 100644
--- a/SimCalorimetry/CaloSimAlgos/interface/CaloShapes.h
+++ b/SimCalorimetry/CaloSimAlgos/interface/CaloShapes.h
@@ -13,7 +13,7 @@ class CaloShapes
   // doesn't take ownership of the pointer
   CaloShapes(const CaloVShape * shape) : theShape(shape) {}
   virtual const CaloVShape * shape(const DetId & detId) const {return theShape;}
-
+  virtual ~CaloShapes() = default;
 private:
   const CaloVShape * theShape;
 };
diff --git a/SimCalorimetry/CaloSimAlgos/interface/CaloVHitCorrection.h b/SimCalorimetry/CaloSimAlgos/interface/CaloVHitCorrection.h
index b0fae0f09fa93..910c902fee9c5 100644
--- a/SimCalorimetry/CaloSimAlgos/interface/CaloVHitCorrection.h
+++ b/SimCalorimetry/CaloSimAlgos/interface/CaloVHitCorrection.h
@@ -9,6 +9,7 @@ namespace CLHEP {
 
 class CaloVHitCorrection {
 public:
+  virtual ~CaloVHitCorrection() = default;
   virtual double delay(const PCaloHit & hit, CLHEP::HepRandomEngine*) const = 0;
 };
 
diff --git a/SimCalorimetry/CaloSimAlgos/interface/CaloVHitFilter.h b/SimCalorimetry/CaloSimAlgos/interface/CaloVHitFilter.h
index b0796d52c4b1b..dc8cbd54ba4b0 100644
--- a/SimCalorimetry/CaloSimAlgos/interface/CaloVHitFilter.h
+++ b/SimCalorimetry/CaloSimAlgos/interface/CaloVHitFilter.h
@@ -6,6 +6,7 @@
 
 class CaloVHitFilter {
 public:
+  virtual ~CaloVHitFilter() = default;
   virtual bool accepts(const PCaloHit & hit) const = 0;
 };
 
diff --git a/SimCalorimetry/CaloSimAlgos/interface/CaloVSimParameterMap.h b/SimCalorimetry/CaloSimAlgos/interface/CaloVSimParameterMap.h
index 62476ecd437c1..02183e615142a 100644
--- a/SimCalorimetry/CaloSimAlgos/interface/CaloVSimParameterMap.h
+++ b/SimCalorimetry/CaloSimAlgos/interface/CaloVSimParameterMap.h
@@ -7,6 +7,7 @@ class CaloSimParameters;
 class CaloVSimParameterMap
 {
 public:
+  virtual ~CaloVSimParameterMap() = default;
   virtual const CaloSimParameters & simParameters(const DetId & id) const = 0;
 };
 
diff --git a/SimCalorimetry/HGCalSimProducers/interface/HGCDigitizerBase.h b/SimCalorimetry/HGCalSimProducers/interface/HGCDigitizerBase.h
index dac79ef00a79f..d456acc2c00c4 100644
--- a/SimCalorimetry/HGCalSimProducers/interface/HGCDigitizerBase.h
+++ b/SimCalorimetry/HGCalSimProducers/interface/HGCDigitizerBase.h
@@ -33,7 +33,6 @@ class HGCDigitizerBase {
      @short CTOR
   */
   HGCDigitizerBase(const edm::ParameterSet &ps); 
-      
  /**
     @short steer digitization mode
  */
@@ -73,7 +72,7 @@ class HGCDigitizerBase {
   /**
      @short DTOR
   */
-  ~HGCDigitizerBase() 
+  virtual ~HGCDigitizerBase() 
     { };
   
   
diff --git a/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoEnergy.h b/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoEnergy.h
index fa94cc294c297..436010b0db79f 100644
--- a/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoEnergy.h
+++ b/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoEnergy.h
@@ -16,6 +16,7 @@
 class HcalZSAlgoEnergy : public HcalZeroSuppressionAlgo {
 public:
   HcalZSAlgoEnergy(bool markAndPass, int level, int start, int samples, bool twosided);
+  virtual ~HcalZSAlgoEnergy() = default;
   void prepare(const HcalDbService* db);
   void done();
 protected:
diff --git a/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoRealistic.h b/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoRealistic.h
index 42572685f39a0..8ae7206b1eeea 100644
--- a/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoRealistic.h
+++ b/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZSAlgoRealistic.h
@@ -15,7 +15,7 @@ class HcalZSAlgoRealistic : public HcalZeroSuppressionAlgo {
 public:
   HcalZSAlgoRealistic(bool markAndPass, std::pair<int,int> HBsearchTS, std::pair<int,int> HEsearchTS, std::pair<int,int> HOsearchTS, std::pair<int,int> HFsearchTS);
   HcalZSAlgoRealistic(bool markAndPass, int levelHB, int levelHE, int levelHO, int levelHF, std::pair<int,int> HBsearchTS, std::pair<int,int> HEsearchTS, std::pair<int,int> HOsearchTS, std::pair<int,int> HFsearchTS);
-  
+  virtual ~HcalZSAlgoRealistic() = default; 
 protected:
   //these need to be overloads instead of templates to avoid linking issues when calling private member function templates
   virtual bool shouldKeep(const HBHEDataFrame& digi) const;
diff --git a/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZeroSuppressionAlgo.h b/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZeroSuppressionAlgo.h
index 0cefed935b139..332a8dcfb5165 100644
--- a/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZeroSuppressionAlgo.h
+++ b/SimCalorimetry/HcalZeroSuppressionProducers/src/HcalZeroSuppressionAlgo.h
@@ -14,6 +14,7 @@ class HcalDbService;
 
 class HcalZeroSuppressionAlgo {
 public:
+  virtual ~HcalZeroSuppressionAlgo() = default;
   void suppress(const HBHEDigiCollection& input, HBHEDigiCollection& output);
   void suppress(const HODigiCollection& input, HODigiCollection& output);
   void suppress(const HFDigiCollection& input, HFDigiCollection& output);