Add const variables and iterators

Framework/Crystal/src/IndexPeaks.cpp

@@ -413,18 +413,18 @@ std::map<std::string, std::string> IndexPeaks::validateInputs() {
   }
 
   // get all runs which have peaksin the table
-  bool commonUB = this->getProperty(Prop::COMMONUB);
+  const bool commonUB = this->getProperty(Prop::COMMONUB);
   if (commonUB) {
-    auto &allPeaks = ws->getPeaks();
+    const auto &allPeaks = ws->getPeaks();
     std::unordered_map<int, int> peaksPerRun;
-    auto it = allPeaks.begin();
-    while (peaksPerRun.size() < 2 && it != allPeaks.end()) {
+    auto it = allPeaks.cbegin();
+    while (peaksPerRun.size() < 2 && it != allPeaks.cend()) {
       peaksPerRun[it->getRunNumber()] = 1;
       ++it;
     };
     if (peaksPerRun.size() < 2) {
       helpMsgs[Prop::COMMONUB] =
-          "CommonUBForAll can only be True if there are peaks from more"
+          "CommonUBForAll can only be True if there are peaks from more "
           "than one run present in the peaks worksapce";
     };
   };