Merge pull request #26553 from makortel/fixGccGeometryTrackerNumberin…

…gBuilder Remove unnecessary calls to vector::empty() and vector::clear() from CmsTrackerPhase{1,2TP}DiskBuilder
cms-sw · Apr 29, 2019 · f7a2ccf · f7a2ccf
2 parents d311c79 + e727819
commit f7a2ccf
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/Geometry/TrackerNumberingBuilder/plugins/CmsTrackerPhase1DiskBuilder.cc b/Geometry/TrackerNumberingBuilder/plugins/CmsTrackerPhase1DiskBuilder.cc
@@ -26,8 +26,6 @@ CmsTrackerPhase1DiskBuilder::PhiPosNegSplit_innerOuter( std::vector< GeometricDe
 
   // now put positive phi (in order) ahead of negative phi as in std geometry
   std::vector<const GeometricDet*> theCompsPosNeg;
-  theCompsPosNeg.empty();
-  theCompsPosNeg.clear();
   // also find the average radius (used to split inner and outer disk panels)
   double theRmin = (**begin).rho();
   double theRmax = theRmin;
@@ -47,8 +45,6 @@ CmsTrackerPhase1DiskBuilder::PhiPosNegSplit_innerOuter( std::vector< GeometricDe
   // force the split radius to be 100 mm to be able to deal with disks with only outer ring
   double radius_split = 100.;
   std::vector<const GeometricDet*> theCompsInnerOuter;
-  theCompsInnerOuter.empty();
-  theCompsInnerOuter.clear();
   unsigned int num_inner = 0;
   for(vector<const GeometricDet*>::const_iterator it=theCompsPosNeg.begin();
       it!=theCompsPosNeg.end();it++){

diff --git a/Geometry/TrackerNumberingBuilder/plugins/CmsTrackerPhase2TPDiskBuilder.cc b/Geometry/TrackerNumberingBuilder/plugins/CmsTrackerPhase2TPDiskBuilder.cc
@@ -26,8 +26,6 @@ CmsTrackerPhase2TPDiskBuilder::PhiPosNegSplit_innerOuter( std::vector< Geometric
 
   // now put positive phi (in order) ahead of negative phi as in std geometry
   std::vector<const GeometricDet*> theCompsPosNeg;
-  theCompsPosNeg.empty();
-  theCompsPosNeg.clear();
   // also find the average radius (used to split inner and outer disk panels)
   double theRmin = (**begin).rho();
   double theRmax = theRmin;
@@ -47,8 +45,6 @@ CmsTrackerPhase2TPDiskBuilder::PhiPosNegSplit_innerOuter( std::vector< Geometric
   // force the split radius to be 100 mm to be able to deal with disks with only outer ring
   double radius_split = 100.;
   std::vector<const GeometricDet*> theCompsInnerOuter;
-  theCompsInnerOuter.empty();
-  theCompsInnerOuter.clear();
   unsigned int num_inner = 0;
   for(vector<const GeometricDet*>::const_iterator it=theCompsPosNeg.begin();
       it!=theCompsPosNeg.end();it++){