From 166c338fde5ad9579167923178beaff631db43cf Mon Sep 17 00:00:00 2001
From: Josh Bendavid <Josh.Bendavid@cern.ch>
Date: Fri, 18 Sep 2015 20:32:29 +0200
Subject: [PATCH] add missing break statement to final protection

---
 RecoLocalCalo/EcalRecAlgos/src/PulseChiSqSNNLS.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/RecoLocalCalo/EcalRecAlgos/src/PulseChiSqSNNLS.cc b/RecoLocalCalo/EcalRecAlgos/src/PulseChiSqSNNLS.cc
index 259cfcc1bed6b..62162524d2212 100644
--- a/RecoLocalCalo/EcalRecAlgos/src/PulseChiSqSNNLS.cc
+++ b/RecoLocalCalo/EcalRecAlgos/src/PulseChiSqSNNLS.cc
@@ -322,6 +322,7 @@ bool PulseChiSqSNNLS::NNLS() {
       //worst case protection
       if (iter>=500) {
         edm::LogWarning("PulseChiSqSNNLS::NNLS()") << "Max Iterations reached at iter " << iter <<  std::endl;
+        break;
       }
       
       //unconstrain parameter