bugfix nullptr check went in an opposite way

cms-sw · May 17, 2021 · a521c64 · a521c64
1 parent 601fbc5
commit a521c64
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/RecoMuon/MuonSeedGenerator/src/RPCSeedOverlapper.cc b/RecoMuon/MuonSeedGenerator/src/RPCSeedOverlapper.cc
@@ -36,7 +36,7 @@ void RPCSeedOverlapper::unsetIO() { isIOset = false; }
 void RPCSeedOverlapper::setGeometry(const RPCGeometry &iGeom) { rpcGeometry = &iGeom; }
 
 void RPCSeedOverlapper::run() {
-  if (isConfigured == false || isIOset == false || rpcGeometry != nullptr) {
+  if (isConfigured == false || isIOset == false || rpcGeometry == nullptr) {
     cout << "Configuration or IO is not set yet" << endl;
     return;
   }
@@ -167,7 +167,7 @@ bool RPCSeedOverlapper::isShareHit(const std::vector<TrackingRecHit const *> &re
   GlobalPoint gpos1 = rpcroll1->toGlobal(lpos1);
   cout << "The hit's position: " << gpos1.x() << ", " << gpos1.y() << ", " << gpos1.z() << endl;
   for (auto const &recHit : recHits) {
-    cout << "Checking the " << n << " th recHit from tempRecHits" << endl;
+    cout << "Checking the " << (n++) << " th recHit from tempRecHits" << endl;
     LocalPoint lpos2 = recHit->localPosition();
     DetId RPCId2 = recHit->geographicalId();
     const GeomDetUnit *rpcroll2 = rpcGeometry.idToDetUnit(RPCId2);