Fix a few more warnings

key4hep · Dec 13, 2023 · ad29a1c · ad29a1c
1 parent f532014
commit ad29a1c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/detector/calorimeter/ECalBarrel_NobleLiquid_InclinedTrapezoids_o1_v01_geo.cpp b/detector/calorimeter/ECalBarrel_NobleLiquid_InclinedTrapezoids_o1_v01_geo.cpp
@@ -574,15 +574,13 @@ static dd4hep::detail::Ref_t createECalBarrelInclined(dd4hep::Detector& aLcdd,
   dd4hep::xml::setDetectorTypeFlag(xmlDetElem, caloDetElem);
   dd4hep::rec::MaterialManager matMgr(envelopeVol);
   dd4hep::rec::LayeredCalorimeterData::Layer caloLayer;
-  double nRadiationLengths   = 0.;
-  double nInteractionLengths = 0.;
 
   double rad_first = Rmin;
   double rad_last = 0;
   double scale_fact = dR / (-Rmin * cos(angle) + sqrt(pow(Rmax, 2) - pow(Rmin * sin(angle), 2)));
   // since the layer height is given along the electrode and not along the radius it needs to be scaled to get the values of layer height radially
   std::cout << "Scaling factor " << scale_fact << std::endl;
-  for (auto il = 0; il < layerHeight.size(); il++) {
+  for (size_t il = 0; il < layerHeight.size(); il++) {
     double thickness_sen = 0.;
     double absorberThickness = 0.;
 
@@ -600,7 +598,7 @@ static dd4hep::detail::Ref_t createECalBarrelInclined(dd4hep::Detector& aLcdd,
     const double value_of_lambda = layerHeight[il] / nInteractionLengths;
     std::string str1("LAr");
 
-    for (auto imat = 0; imat < materials.size(); imat++) {
+    for (size_t imat = 0; imat < materials.size(); imat++) {
 
       std::string str2(materials.at(imat).first.name());
       if (str1.compare(str2) == 0){

diff --git a/detector/calorimeter/SECalEndcap_o1_v01_geo.cpp b/detector/calorimeter/SECalEndcap_o1_v01_geo.cpp
@@ -193,7 +193,6 @@ static Ref_t create_detector(Detector& theDetector, xml_h e, SensitiveDetector s
       // Loop over the sets of layer elements in the detector
       double check_thick = 0.0;
       int l_num = 1;
-      int l_set = 0;
       for(xml_coll_t li(x_det,_U(layer)); li; ++li)  {
 
 	xml_comp_t x_layer = li;