Merge pull request #45491 from smuzaffar/ambiguous-reversed-operator-DTSegmentCand

DTSegmentCand: Fix ambiguous-reversed-operator warning by using const operator

Author: cmsbuild

RecoLocalMuon/DTSegment/src/DTSegmentCand.cc

@@ -34,7 +34,7 @@ DTSegmentCand::DTSegmentCand(const AssPointCont& hits,
 DTSegmentCand::~DTSegmentCand() {}
 
 /* Operations */
-bool DTSegmentCand::operator==(const DTSegmentCand& seg) {
+bool DTSegmentCand::operator==(const DTSegmentCand& seg) const {
   static const double epsilon = 0.00001;
   if (nHits() != seg.nHits())
     return false;
@@ -49,7 +49,7 @@ bool DTSegmentCand::operator==(const DTSegmentCand& seg) {
   return true;
 }
 
-bool DTSegmentCand::operator<(const DTSegmentCand& seg) {
+bool DTSegmentCand::operator<(const DTSegmentCand& seg) const {
   if (nHits() == seg.nHits())
     return (chi2() > seg.chi2());
   return (nHits() < seg.nHits());

RecoLocalMuon/DTSegment/src/DTSegmentCand.h

@@ -67,10 +67,10 @@ class DTSegmentCand {
   virtual double t0() const { return thet0; }
 
   /// equality operator based on position, direction, chi2 and nHits
-  virtual bool operator==(const DTSegmentCand& seg);
+  virtual bool operator==(const DTSegmentCand& seg) const;
 
   /// less operator based on nHits and chi2
-  virtual bool operator<(const DTSegmentCand& seg);
+  virtual bool operator<(const DTSegmentCand& seg) const;
 
   /// the super layer on which relies
   const DTSuperLayer* superLayer() const { return theSL; }