Use default op= in LASModuleProfile

This avoids a clang warning about deprecated copy constructor.
cms-sw · Mar 19, 2024 · 21833f1 · 21833f1
1 parent 28f76ff
commit 21833f1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/Alignment/LaserAlignment/interface/LASModuleProfile.h b/Alignment/LaserAlignment/interface/LASModuleProfile.h
@@ -20,7 +20,6 @@ class LASModuleProfile {
   void SetValue(unsigned int theStripNumber, const double& theValue) { data.at(theStripNumber) = theValue; }
   void SetAllValuesTo(const double&);
   void DumpToArray(double[512]);
-  LASModuleProfile& operator=(const LASModuleProfile&);
   LASModuleProfile operator+(const LASModuleProfile&);
   LASModuleProfile operator-(const LASModuleProfile&);
   LASModuleProfile operator+(const double[512]);

diff --git a/Alignment/LaserAlignment/src/LASModuleProfile.cc b/Alignment/LaserAlignment/src/LASModuleProfile.cc
@@ -100,21 +100,6 @@ void LASModuleProfile::Init(void) {
   data.resize(512);
 }
 
-LASModuleProfile& LASModuleProfile::operator=(const LASModuleProfile& anotherProfile) {
-  ///
-  ///
-  ///
-
-  // check for self-assignment
-  if (this != &anotherProfile) {
-    for (unsigned int i = 0; i < 512; ++i) {
-      data.at(i) = anotherProfile.GetValue(i);
-    }
-  }
-
-  return *this;
-}
-
 LASModuleProfile LASModuleProfile::operator+(const LASModuleProfile& anotherProfile) {
   ///
   ///