From ffc3ee41f509a6755ea1fd6b1a7a2cb455bd0cee Mon Sep 17 00:00:00 2001
From: Samuel Jackson <samueljackson@outlook.com>
Date: Wed, 4 Oct 2017 13:18:18 +0100
Subject: [PATCH] Apply clang format patch

---
 .../Algorithms/src/Bin2DPowderDiffraction.cpp      |  2 +-
 Framework/Algorithms/src/ClearCache.cpp            |  2 +-
 Framework/Algorithms/src/ConvertAxisByFormula.cpp  |  6 +++---
 .../Algorithms/src/ReflectometryReductionOne2.cpp  |  2 +-
 .../src/ReflectometryReductionOneAuto2.cpp         |  2 +-
 .../RunCombinationHelpers/SampleLogsBehaviour.cpp  |  2 +-
 Framework/Kernel/src/CompositeValidator.cpp        |  2 +-
 .../api/src/FitFunctions/IFunctionAdapter.cpp      |  2 +-
 Framework/SINQ/src/PoldiFitPeaks1D2.cpp            |  2 +-
 MantidPlot/src/ConfigDialog.cpp                    | 14 +++++++-------
 MantidPlot/src/Mantid/PeakPickerTool.cpp           |  2 +-
 MantidPlot/src/ProjectSaveView.cpp                 | 12 ++++++------
 .../DynamicPDF/DPDFFitControl.cpp                  |  2 +-
 .../EnggDiffraction/EnggDiffFittingPresenter.cpp   |  6 +++---
 .../EnggDiffraction/EnggDiffractionPresenter.cpp   |  2 +-
 .../Indirect/IndirectDiffractionReduction.cpp      |  2 +-
 .../Indirect/IndirectMolDyn.cpp                    |  2 +-
 .../Muon/ALCDataLoadingView.cpp                    |  2 +-
 .../Muon/MuonAnalysisDataLoader.cpp                |  4 ++--
 .../Muon/MuonAnalysisFitDataPresenter.cpp          |  2 +-
 .../Muon/MuonAnalysisHelper.cpp                    |  2 +-
 qt/widgets/common/src/FunctionBrowser.cpp          |  2 +-
 qt/widgets/common/src/MuonFitPropertyBrowser.cpp   | 10 +++++-----
 .../instrumentview/src/InstrumentWidgetTreeTab.cpp |  2 +-
 .../instrumentview/src/Shape2DCollection.cpp       |  2 +-
 qt/widgets/sliceviewer/src/PeaksViewer.cpp         |  4 ++--
 qt/widgets/sliceviewer/src/SliceViewer.cpp         |  2 +-
 qt/widgets/spectrumviewer/src/SpectrumView.cpp     |  4 ++--
 28 files changed, 50 insertions(+), 50 deletions(-)

diff --git a/Framework/Algorithms/src/Bin2DPowderDiffraction.cpp b/Framework/Algorithms/src/Bin2DPowderDiffraction.cpp
index fdc1bdf12a9..082a1690e8b 100644
--- a/Framework/Algorithms/src/Bin2DPowderDiffraction.cpp
+++ b/Framework/Algorithms/src/Bin2DPowderDiffraction.cpp
@@ -183,7 +183,7 @@ MatrixWorkspace_sptr Bin2DPowderDiffraction::createOutputWorkspace() {
                   << std::endl;
 
     size_t idx = 0;
-    for (const auto& Xbins : fileXbins) {
+    for (const auto &Xbins : fileXbins) {
       g_log.debug() << "Xbins size: " << Xbins.size() << std::endl;
       BinEdges binEdges(Xbins);
       outputWS->setBinEdges(idx, binEdges);
diff --git a/Framework/Algorithms/src/ClearCache.cpp b/Framework/Algorithms/src/ClearCache.cpp
index ff66b18a531..ca2702cd3eb 100644
--- a/Framework/Algorithms/src/ClearCache.cpp
+++ b/Framework/Algorithms/src/ClearCache.cpp
@@ -136,7 +136,7 @@ int ClearCache::deleteFiles(const std::string &path,
   std::set<std::string> files;
   Poco::Glob::glob(pathPattern, files, Poco::Glob::GLOB_CASELESS);
 
-  for (const auto& filepath : files) {
+  for (const auto &filepath : files) {
     Poco::File file(filepath);
     g_log.debug("Deleting file " + filepath);
     try {
diff --git a/Framework/Algorithms/src/ConvertAxisByFormula.cpp b/Framework/Algorithms/src/ConvertAxisByFormula.cpp
index 68f45db705b..05a165a5428 100644
--- a/Framework/Algorithms/src/ConvertAxisByFormula.cpp
+++ b/Framework/Algorithms/src/ConvertAxisByFormula.cpp
@@ -150,7 +150,7 @@ void ConvertAxisByFormula::exec() {
   mu::Parser p;
   try {
     // set parameter lookups for the axis value
-    for (const auto& variable : variables) {
+    for (const auto &variable : variables) {
       p.DefineVar(variable->name, &(variable->value));
     }
     // set some constants
@@ -262,7 +262,7 @@ void ConvertAxisByFormula::exec() {
 
 void ConvertAxisByFormula::setAxisValue(const double &value,
                                         std::vector<Variable_ptr> &variables) {
-  for (const auto& variable : variables) {
+  for (const auto &variable : variables) {
     if (!variable->isGeometric) {
       variable->value = value;
     }
@@ -281,7 +281,7 @@ void ConvertAxisByFormula::calculateValues(
 void ConvertAxisByFormula::setGeometryValues(
     const API::SpectrumInfo &specInfo, const size_t index,
     std::vector<Variable_ptr> &variables) {
-  for (const auto& variable : variables) {
+  for (const auto &variable : variables) {
     if (variable->isGeometric) {
       if (variable->name == "twotheta") {
         variable->value = specInfo.twoTheta(index);
diff --git a/Framework/Algorithms/src/ReflectometryReductionOne2.cpp b/Framework/Algorithms/src/ReflectometryReductionOne2.cpp
index ecfaaba6658..ccc851035de 100644
--- a/Framework/Algorithms/src/ReflectometryReductionOne2.cpp
+++ b/Framework/Algorithms/src/ReflectometryReductionOne2.cpp
@@ -94,7 +94,7 @@ void translateAdd(const std::string &instructions,
 
   std::vector<size_t> outSpectra;
   outSpectra.reserve(spectra.count());
-  for (const auto& spectrum : spectra) {
+  for (const auto &spectrum : spectra) {
     // add this spectrum to the group we're about to add
     outSpectra.push_back(boost::lexical_cast<size_t>(spectrum));
   }
diff --git a/Framework/Algorithms/src/ReflectometryReductionOneAuto2.cpp b/Framework/Algorithms/src/ReflectometryReductionOneAuto2.cpp
index 29c367d683f..170e36f77af 100644
--- a/Framework/Algorithms/src/ReflectometryReductionOneAuto2.cpp
+++ b/Framework/Algorithms/src/ReflectometryReductionOneAuto2.cpp
@@ -346,7 +346,7 @@ std::vector<std::string> ReflectometryReductionOneAuto2::getDetectorNames(
   std::vector<std::string> detectors;
 
   try {
-    for (const auto& wsIndex : wsIndices) {
+    for (const auto &wsIndex : wsIndices) {
 
       size_t index = boost::lexical_cast<size_t>(wsIndex);
 
diff --git a/Framework/Algorithms/src/RunCombinationHelpers/SampleLogsBehaviour.cpp b/Framework/Algorithms/src/RunCombinationHelpers/SampleLogsBehaviour.cpp
index 37287a54b10..8d7f330f298 100644
--- a/Framework/Algorithms/src/RunCombinationHelpers/SampleLogsBehaviour.cpp
+++ b/Framework/Algorithms/src/RunCombinationHelpers/SampleLogsBehaviour.cpp
@@ -676,7 +676,7 @@ void SampleLogsBehaviour::removeSampleLogsFromWorkspace(
  * @param addeeWS the workspace being merged
  */
 void SampleLogsBehaviour::readdSampleLogToWorkspace(MatrixWorkspace &addeeWS) {
-  for (const auto& item : m_addeeLogMap) {
+  for (const auto &item : m_addeeLogMap) {
     auto property = std::unique_ptr<Kernel::Property>(item->clone());
     addeeWS.mutableRun().addProperty(std::move(property));
   }
diff --git a/Framework/Kernel/src/CompositeValidator.cpp b/Framework/Kernel/src/CompositeValidator.cpp
index d619f7672a1..b6af12842ca 100644
--- a/Framework/Kernel/src/CompositeValidator.cpp
+++ b/Framework/Kernel/src/CompositeValidator.cpp
@@ -82,7 +82,7 @@ std::string CompositeValidator::check(const boost::any &value) const {
 }
 
 std::string CompositeValidator::checkAll(const boost::any &value) const {
-  for (const auto& validator : m_children) {
+  for (const auto &validator : m_children) {
     const auto error = validator->check(value);
     // exit on the first error, to avoid passing doing more tests on invalid
     // objects that could fail
diff --git a/Framework/PythonInterface/mantid/api/src/FitFunctions/IFunctionAdapter.cpp b/Framework/PythonInterface/mantid/api/src/FitFunctions/IFunctionAdapter.cpp
index 2f3e777f21f..8f1398c9ae6 100644
--- a/Framework/PythonInterface/mantid/api/src/FitFunctions/IFunctionAdapter.cpp
+++ b/Framework/PythonInterface/mantid/api/src/FitFunctions/IFunctionAdapter.cpp
@@ -183,7 +183,7 @@ boost::python::object
 IFunctionAdapter::createPythonEquivalentFunctions(IFunction &self) {
   auto functions = self.createEquivalentFunctions();
   boost::python::list list;
-  for (const auto& fun : functions) {
+  for (const auto &fun : functions) {
     list.append(fun);
   }
   return list;
diff --git a/Framework/SINQ/src/PoldiFitPeaks1D2.cpp b/Framework/SINQ/src/PoldiFitPeaks1D2.cpp
index ef70edf5def..8ca44f3a4e1 100644
--- a/Framework/SINQ/src/PoldiFitPeaks1D2.cpp
+++ b/Framework/SINQ/src/PoldiFitPeaks1D2.cpp
@@ -283,7 +283,7 @@ PoldiFitPeaks1D2::fitPeaks(const PoldiPeakCollection_sptr &peaks) {
   Workspace2D_sptr dataWorkspace = getProperty("InputWorkspace");
   m_fitplots->removeAll();
 
-  for (const auto& currentRange : reducedRanges) {
+  for (const auto &currentRange : reducedRanges) {
     int nMin = getBestChebyshevPolynomialDegree(dataWorkspace, currentRange);
 
     if (nMin > -1) {
diff --git a/MantidPlot/src/ConfigDialog.cpp b/MantidPlot/src/ConfigDialog.cpp
index 9d4481d99c6..41bcdf18215 100644
--- a/MantidPlot/src/ConfigDialog.cpp
+++ b/MantidPlot/src/ConfigDialog.cpp
@@ -716,7 +716,7 @@ void ConfigDialog::initMantidPage() {
 
   // Populate boxes
   auto faclist = cfgSvc.getFacilityNames();
-  for (const auto& it : faclist) {
+  for (const auto &it : faclist) {
     facility->addItem(QString::fromStdString(it));
   }
 
@@ -1197,7 +1197,7 @@ void ConfigDialog::populateProgramTree() {
 
 void ConfigDialog::updateProgramTree() {
   // Store into a map ready to go into config service when apply is clicked
-  for (const auto& itr : m_sendToSettings) {
+  for (const auto &itr : m_sendToSettings) {
     // creating the map of kvps needs to happen first as createing the item
     // requires them.
     std::map<std::string, std::string> programKeysAndDetails = itr.second;
@@ -1223,7 +1223,7 @@ void ConfigDialog::updateChildren(
     QTreeWidgetItem *program) {
   program->takeChildren();
   // get the current program's (itr) keys and values (pItr)
-  for (const auto& pItr : programKeysAndDetails) {
+  for (const auto &pItr : programKeysAndDetails) {
     QTreeWidgetItem *item = new QTreeWidgetItem(program);
     QString itemText = QString("   ")
                            .append(QString::fromStdString(pItr.first))
@@ -1242,7 +1242,7 @@ void ConfigDialog::updateSendToTab() {
   std::vector<std::string> programNames =
       cfgSvc.getKeys("workspace.sendto.name");
 
-  for (const auto& itr : m_sendToSettings) {
+  for (const auto &itr : m_sendToSettings) {
     for (size_t i = 0; i < programNames.size(); ++i) {
       if (programNames[i] == itr.first) {
         // The selected program hasn't been deleted so set to blank string (all
@@ -1255,7 +1255,7 @@ void ConfigDialog::updateSendToTab() {
 
     std::map<std::string, std::string> programKeysAndDetails = itr.second;
 
-    for (const auto& pItr : programKeysAndDetails) {
+    for (const auto &pItr : programKeysAndDetails) {
       if (pItr.second != "")
         cfgSvc.setString("workspace.sendto." + itr.first + "." + pItr.first,
                          pItr.second);
@@ -1289,7 +1289,7 @@ void ConfigDialog::refreshTreeCategories() {
   widgetMap categories; // Keeps track of categories added to the tree
 
   // Loop over all categories loaded into Mantid
-  for (const auto& i : categoryMap) {
+  for (const auto &i : categoryMap) {
 
     QString catNames = QString::fromStdString(i.first);
     // Start recursion down building tree from names
@@ -2872,7 +2872,7 @@ QStringList ConfigDialog::buildHiddenCategoryString(QTreeWidgetItem *parent) {
     }
 
     QStringList childResults = buildHiddenCategoryString(item);
-    for (const auto& it : childResults) {
+    for (const auto &it : childResults) {
       results.append(item->text(0) + "\\" + it);
     }
   }
diff --git a/MantidPlot/src/Mantid/PeakPickerTool.cpp b/MantidPlot/src/Mantid/PeakPickerTool.cpp
index 45f5424c28f..718608590cf 100644
--- a/MantidPlot/src/Mantid/PeakPickerTool.cpp
+++ b/MantidPlot/src/Mantid/PeakPickerTool.cpp
@@ -1033,7 +1033,7 @@ bool PeakPickerTool::initializeFromCurve(PlotCurve *curve) {
  */
 void PeakPickerTool::addExistingFitsAndGuess(const QStringList &curvesList) {
   bool hasGuess = false;
-  for (const auto& curveName : curvesList) {
+  for (const auto &curveName : curvesList) {
     if (curveName.contains(QRegExp("Workspace-[Calc|Diff]"))) { // fit
       m_curveNames.append(curveName);
     } else if (curveName == "CompositeFunction") { // guess
diff --git a/MantidPlot/src/ProjectSaveView.cpp b/MantidPlot/src/ProjectSaveView.cpp
index 3e0730a493d..2e661c1ca0c 100644
--- a/MantidPlot/src/ProjectSaveView.cpp
+++ b/MantidPlot/src/ProjectSaveView.cpp
@@ -91,7 +91,7 @@ void ProjectSaveView::setProjectPath(const QString &path) {
 void ProjectSaveView::updateWorkspacesList(
     const std::vector<WorkspaceInfo> &workspaces) {
   m_ui.workspaceList->clear();
-  for (const auto& info : workspaces) {
+  for (const auto &info : workspaces) {
     addWorkspaceItem(info);
   }
   // pad the header for longish workspace names
@@ -108,7 +108,7 @@ void ProjectSaveView::updateWorkspacesList(
 void ProjectSaveView::updateIncludedWindowsList(
     const std::vector<WindowInfo> &windows) {
   m_ui.includedWindows->clear();
-  for (const auto& info : windows) {
+  for (const auto &info : windows) {
     addWindowItem(m_ui.includedWindows, info);
   }
 
@@ -125,7 +125,7 @@ void ProjectSaveView::updateIncludedWindowsList(
 void ProjectSaveView::updateExcludedWindowsList(
     const std::vector<WindowInfo> &windows) {
   m_ui.excludedWindows->clear();
-  for (const auto& info : windows) {
+  for (const auto &info : windows) {
     addWindowItem(m_ui.excludedWindows, info);
   }
 
@@ -138,7 +138,7 @@ void ProjectSaveView::updateExcludedWindowsList(
  */
 void ProjectSaveView::removeFromIncludedWindowsList(
     const std::vector<std::string> &windows) {
-  for (const auto& name : windows) {
+  for (const auto &name : windows) {
     removeItem(m_ui.includedWindows, name);
   }
 }
@@ -149,7 +149,7 @@ void ProjectSaveView::removeFromIncludedWindowsList(
  */
 void ProjectSaveView::removeFromExcludedWindowsList(
     const std::vector<std::string> &windows) {
-  for (const auto& name : windows) {
+  for (const auto &name : windows) {
     removeItem(m_ui.excludedWindows, name);
   }
 }
@@ -312,7 +312,7 @@ void ProjectSaveView::addWindowItem(QTreeWidget *widget,
 void ProjectSaveView::addWorkspaceItem(const WorkspaceInfo &info) {
   auto item = makeWorkspaceItem(info);
 
-  for (const auto& subInfo : info.subWorkspaces) {
+  for (const auto &subInfo : info.subWorkspaces) {
     auto subItem = makeWorkspaceItem(subInfo);
     item->addChild(subItem);
   }
diff --git a/qt/scientific_interfaces/DynamicPDF/DPDFFitControl.cpp b/qt/scientific_interfaces/DynamicPDF/DPDFFitControl.cpp
index 8621e73dbe8..23845d0cfea 100644
--- a/qt/scientific_interfaces/DynamicPDF/DPDFFitControl.cpp
+++ b/qt/scientific_interfaces/DynamicPDF/DPDFFitControl.cpp
@@ -360,7 +360,7 @@ void FitControl::saveBuiltInModels() {
                             "Quadratic,A0=0,A1=0,A2=0;name=Gaussian,Height=0,"
                             "PeakCentre=0,Sigma=0);name=LinearBackground,A0=0,"
                             "A1=0";
-  for (const auto& modelName : models.keys()) {
+  for (const auto &modelName : models.keys()) {
     settings.setValue(modelName, models[modelName]);
   }
 }
diff --git a/qt/scientific_interfaces/EnggDiffraction/EnggDiffFittingPresenter.cpp b/qt/scientific_interfaces/EnggDiffraction/EnggDiffFittingPresenter.cpp
index e799cc191b3..71c4dea80a2 100644
--- a/qt/scientific_interfaces/EnggDiffraction/EnggDiffFittingPresenter.cpp
+++ b/qt/scientific_interfaces/EnggDiffraction/EnggDiffFittingPresenter.cpp
@@ -677,7 +677,7 @@ EnggDiffFittingPresenter::enableMultiRun(const std::string &firstRun,
 
   bool foundAllRuns = true;
 
-  for (const auto& runNumber : RunNumberVec) {
+  for (const auto &runNumber : RunNumberVec) {
     // Get full path for every run selected
     std::vector<std::string> foundFileNames;
     if (findFilePathFromBaseName(workingDirectory, runNumber, foundFileNames)) {
@@ -760,7 +760,7 @@ void EnggDiffFittingPresenter::processFitAllPeaks() {
   const std::string fitPeaksData = validateFittingexpectedPeaks(fittingPeaks);
 
   g_log.debug() << "Focused files found are: " << fitPeaksData << '\n';
-  for (const auto& dir : g_multi_run_directories) {
+  for (const auto &dir : g_multi_run_directories) {
     g_log.debug() << dir << '\n';
   }
 
@@ -1599,7 +1599,7 @@ void EnggDiffFittingPresenter::setBankItems(
   try {
     // Keep track of current loop iteration for banks
     int index = 0;
-    for (const auto& filePath : bankFiles) {
+    for (const auto &filePath : bankFiles) {
 
       const Poco::Path bankFile(filePath);
       const std::string strVecFile = bankFile.toString();
diff --git a/qt/scientific_interfaces/EnggDiffraction/EnggDiffractionPresenter.cpp b/qt/scientific_interfaces/EnggDiffraction/EnggDiffractionPresenter.cpp
index 6bdfac3005d..4288f92e418 100644
--- a/qt/scientific_interfaces/EnggDiffraction/EnggDiffractionPresenter.cpp
+++ b/qt/scientific_interfaces/EnggDiffraction/EnggDiffractionPresenter.cpp
@@ -729,7 +729,7 @@ std::vector<std::string> EnggDiffractionPresenter::isValidMultiRunNumber(
   if (paths.empty() || paths.front().empty())
     return multi_run_number;
 
-  for (const auto& path : paths) {
+  for (const auto &path : paths) {
     std::string run_number;
     try {
       if (Poco::File(path).exists()) {
diff --git a/qt/scientific_interfaces/Indirect/IndirectDiffractionReduction.cpp b/qt/scientific_interfaces/Indirect/IndirectDiffractionReduction.cpp
index 932d3599d40..dab707af3b1 100644
--- a/qt/scientific_interfaces/Indirect/IndirectDiffractionReduction.cpp
+++ b/qt/scientific_interfaces/Indirect/IndirectDiffractionReduction.cpp
@@ -229,7 +229,7 @@ void IndirectDiffractionReduction::plotResults() {
  * Handles saving the reductions from the generic algorithm.
  */
 void IndirectDiffractionReduction::saveReductions() {
-  for (const auto& wsName : m_plotWorkspaces) {
+  for (const auto &wsName : m_plotWorkspaces) {
     const auto workspaceExists =
         AnalysisDataService::Instance().doesExist(wsName);
     if (workspaceExists) {
diff --git a/qt/scientific_interfaces/Indirect/IndirectMolDyn.cpp b/qt/scientific_interfaces/Indirect/IndirectMolDyn.cpp
index 2383ba7e18a..c8af0075019 100644
--- a/qt/scientific_interfaces/Indirect/IndirectMolDyn.cpp
+++ b/qt/scientific_interfaces/Indirect/IndirectMolDyn.cpp
@@ -136,7 +136,7 @@ void IndirectMolDyn::plotClicked() {
     auto names = diffResultsGroup->getNames();
     auto plotType = m_uiForm.cbPlot->currentText();
 
-    for (const auto& wsName : names) {
+    for (const auto &wsName : names) {
       if (plotType == "Spectra" || plotType == "Both")
         plotSpectrum(QString::fromStdString(wsName));
 
diff --git a/qt/scientific_interfaces/Muon/ALCDataLoadingView.cpp b/qt/scientific_interfaces/Muon/ALCDataLoadingView.cpp
index fdb2296e862..47cfb94d1bb 100644
--- a/qt/scientific_interfaces/Muon/ALCDataLoadingView.cpp
+++ b/qt/scientific_interfaces/Muon/ALCDataLoadingView.cpp
@@ -229,7 +229,7 @@ void ALCDataLoadingView::setAvailableItems(
   }
 
   // Add new items
-  for (const auto& item : items) {
+  for (const auto &item : items) {
     if (item != previousValue) { // has already been added
       comboBox->addItem(QString::fromStdString(item));
     }
diff --git a/qt/scientific_interfaces/Muon/MuonAnalysisDataLoader.cpp b/qt/scientific_interfaces/Muon/MuonAnalysisDataLoader.cpp
index 5240031851b..e3761665e59 100644
--- a/qt/scientific_interfaces/Muon/MuonAnalysisDataLoader.cpp
+++ b/qt/scientific_interfaces/Muon/MuonAnalysisDataLoader.cpp
@@ -76,7 +76,7 @@ LoadResult MuonAnalysisDataLoader::loadFiles(const QStringList &files) const {
   const auto toString = [](QStringList qsl) {
     std::ostringstream oss;
     qsl.sort();
-    for (const QString& qs : qsl) {
+    for (const QString &qs : qsl) {
       oss << qs.toStdString() << ",";
     }
     return oss.str();
@@ -98,7 +98,7 @@ LoadResult MuonAnalysisDataLoader::loadFiles(const QStringList &files) const {
   std::string instrName; // Instrument name all the run files should belong to
 
   // Go through all the files and try to load them
-  for (const auto& fileName : files) {
+  for (const auto &fileName : files) {
     std::string file = fileName.toStdString();
 
     // Set up load algorithm
diff --git a/qt/scientific_interfaces/Muon/MuonAnalysisFitDataPresenter.cpp b/qt/scientific_interfaces/Muon/MuonAnalysisFitDataPresenter.cpp
index af4b84c6205..8518ad77767 100644
--- a/qt/scientific_interfaces/Muon/MuonAnalysisFitDataPresenter.cpp
+++ b/qt/scientific_interfaces/Muon/MuonAnalysisFitDataPresenter.cpp
@@ -344,7 +344,7 @@ std::vector<std::string> MuonAnalysisFitDataPresenter::generateWorkspaceNames(
     }
   }
 
-  for (const auto& runsVector : runNumberVectors) {
+  for (const auto &runsVector : runNumberVectors) {
     params.runs = runsVector;
     for (const auto &group : groups) {
       params.itemType = getItemType(group.toStdString());
diff --git a/qt/scientific_interfaces/Muon/MuonAnalysisHelper.cpp b/qt/scientific_interfaces/Muon/MuonAnalysisHelper.cpp
index d5d371d5780..9bc0bf7a5f9 100644
--- a/qt/scientific_interfaces/Muon/MuonAnalysisHelper.cpp
+++ b/qt/scientific_interfaces/Muon/MuonAnalysisHelper.cpp
@@ -731,7 +731,7 @@ std::pair<std::string, std::string> findLogRange(
     const std::vector<Workspace_sptr> &workspaces, const std::string &logName,
     bool (*isLessThan)(const std::string &first, const std::string &second)) {
   std::string smallest, largest;
-  for (const auto& ws : workspaces) {
+  for (const auto &ws : workspaces) {
     auto range = findLogRange(ws, logName, isLessThan);
     if (smallest.empty() || isLessThan(range.first, smallest)) {
       smallest = range.first;
diff --git a/qt/widgets/common/src/FunctionBrowser.cpp b/qt/widgets/common/src/FunctionBrowser.cpp
index 75028395094..9cdf0a13470 100644
--- a/qt/widgets/common/src/FunctionBrowser.cpp
+++ b/qt/widgets/common/src/FunctionBrowser.cpp
@@ -1551,7 +1551,7 @@ FunctionBrowser::getParameterProperty(const QString &funcIndex,
  */
 void FunctionBrowser::updateParameters(const Mantid::API::IFunction &fun) {
   const auto paramNames = fun.getParameterNames();
-  for (const auto& parameter : paramNames) {
+  for (const auto &parameter : paramNames) {
     const QString qName = QString::fromStdString(parameter);
     setParameter(qName, fun.getParameter(parameter));
     const size_t index = fun.parameterIndex(parameter);
diff --git a/qt/widgets/common/src/MuonFitPropertyBrowser.cpp b/qt/widgets/common/src/MuonFitPropertyBrowser.cpp
index 917a7fbc8e9..645a203a8f1 100644
--- a/qt/widgets/common/src/MuonFitPropertyBrowser.cpp
+++ b/qt/widgets/common/src/MuonFitPropertyBrowser.cpp
@@ -1262,7 +1262,7 @@ void MuonFitPropertyBrowser::setAvailableGroups(const QStringList &groups) {
   }
   clearGroupCheckboxes();
   QSettings settings;
-  for (const auto& group : groups) {
+  for (const auto &group : groups) {
     addGroupCheckbox(group);
   }
 }
@@ -1336,7 +1336,7 @@ void MuonFitPropertyBrowser::setAllGroups() {
   clearChosenGroups();
   for (auto iter = m_groupBoxes.constBegin(); iter != m_groupBoxes.constEnd();
        ++iter) {
-    for (const auto& group : m_groupsList) {
+    for (const auto &group : m_groupsList) {
       if (iter.key().toStdString() == group) {
         m_boolManager->setValue(iter.value(), true);
       }
@@ -1351,7 +1351,7 @@ void MuonFitPropertyBrowser::setAllPairs() {
   for (auto iter = m_groupBoxes.constBegin(); iter != m_groupBoxes.constEnd();
        ++iter) {
     bool isItGroup = false;
-    for (const auto& group : m_groupsList) {
+    for (const auto &group : m_groupsList) {
       if (iter.key().toStdString() == group) {
         isItGroup = true;
       }
@@ -1449,7 +1449,7 @@ void MuonFitPropertyBrowser::setAvailablePeriods(const QStringList &periods) {
 
   clearPeriodCheckboxes();
 
-  for (const auto& group : periods) {
+  for (const auto &group : periods) {
     addPeriodCheckbox(group);
   }
 }
@@ -1524,7 +1524,7 @@ QStringList MuonFitPropertyBrowser::getChosenPeriods() const {
 void MuonFitPropertyBrowser::setChosenPeriods(
     const QStringList &chosenPeriods) {
   clearChosenPeriods();
-  for (const auto& selected : chosenPeriods) {
+  for (const auto &selected : chosenPeriods) {
     for (auto iter = m_periodBoxes.constBegin();
          iter != m_periodBoxes.constEnd(); ++iter) {
       auto tmp = iter.key();
diff --git a/qt/widgets/instrumentview/src/InstrumentWidgetTreeTab.cpp b/qt/widgets/instrumentview/src/InstrumentWidgetTreeTab.cpp
index 2348398c9f3..c703547ff0b 100644
--- a/qt/widgets/instrumentview/src/InstrumentWidgetTreeTab.cpp
+++ b/qt/widgets/instrumentview/src/InstrumentWidgetTreeTab.cpp
@@ -107,7 +107,7 @@ std::string InstrumentWidgetTreeTab::saveToProject() const {
 
   auto names = m_instrumentTree->findExpandedComponents();
   tab.writeLine("ExpandedItems");
-  for (const auto& name : names) {
+  for (const auto &name : names) {
     tab << name;
   }
 
diff --git a/qt/widgets/instrumentview/src/Shape2DCollection.cpp b/qt/widgets/instrumentview/src/Shape2DCollection.cpp
index 8e13236a4f3..9221f4d52d2 100644
--- a/qt/widgets/instrumentview/src/Shape2DCollection.cpp
+++ b/qt/widgets/instrumentview/src/Shape2DCollection.cpp
@@ -741,7 +741,7 @@ void Shape2DCollection::eraseFree(const QPolygonF &polygon) {
  */
 void Shape2DCollection::loadFromProject(const std::string &lines) {
   API::TSVSerialiser tsv(lines);
-  for (const auto& shapeLines : tsv.sections("shape")) {
+  for (const auto &shapeLines : tsv.sections("shape")) {
     Shape2D *shape = Shape2D::loadFromProject(shapeLines);
     m_shapes.push_back(shape);
     emit shapeCreated();
diff --git a/qt/widgets/sliceviewer/src/PeaksViewer.cpp b/qt/widgets/sliceviewer/src/PeaksViewer.cpp
index 3e47ce81ebb..22d539db6aa 100644
--- a/qt/widgets/sliceviewer/src/PeaksViewer.cpp
+++ b/qt/widgets/sliceviewer/src/PeaksViewer.cpp
@@ -197,7 +197,7 @@ void PeaksViewer::loadFromProject(const std::string &lines) {
     return;
 
   // load presented workspaces
-  for (const auto& section : tsv.sections("peaksworkspace"))
+  for (const auto &section : tsv.sections("peaksworkspace"))
     loadPresentedWorkspace(section);
 
   // Apply zooming/peak selection
@@ -280,7 +280,7 @@ std::string PeaksViewer::saveToProject() const {
 
   // save all workspaces
   auto workspaces = m_presenter->presentedWorkspaces();
-  for (const auto& ws : workspaces)
+  for (const auto &ws : workspaces)
     tsv.writeSection("peaksworkspace", savePresentedWorkspace(ws));
 
   // save zoom a particular peak
diff --git a/qt/widgets/sliceviewer/src/SliceViewer.cpp b/qt/widgets/sliceviewer/src/SliceViewer.cpp
index 8cbec86f510..721f8801d61 100644
--- a/qt/widgets/sliceviewer/src/SliceViewer.cpp
+++ b/qt/widgets/sliceviewer/src/SliceViewer.cpp
@@ -2881,7 +2881,7 @@ void SliceViewer::loadFromProject(const std::string &lines) {
     API::TSVSerialiser peaks(peaksViewerLines);
 
     QStringList names;
-    for (const auto& section : peaks.sections("peaksworkspace")) {
+    for (const auto &section : peaks.sections("peaksworkspace")) {
       API::TSVSerialiser sec(section);
       QString name;
 
diff --git a/qt/widgets/spectrumviewer/src/SpectrumView.cpp b/qt/widgets/spectrumviewer/src/SpectrumView.cpp
index 4bd6d1d050c..85a2bc01625 100644
--- a/qt/widgets/spectrumviewer/src/SpectrumView.cpp
+++ b/qt/widgets/spectrumviewer/src/SpectrumView.cpp
@@ -367,7 +367,7 @@ std::string SpectrumView::saveToProject(ApplicationWindow *app) {
     spec.writeLine("ColorMapFileName") << colorMapFileName;
 
   spec.writeLine("Workspaces");
-  for (const auto& source : m_dataSource) {
+  for (const auto &source : m_dataSource) {
     spec << source->getWorkspace()->getName();
   }
 
@@ -396,7 +396,7 @@ std::string SpectrumView::getWindowName() {
 
 std::vector<std::string> SpectrumView::getWorkspaceNames() {
   std::vector<std::string> names;
-  for (const auto& source : m_dataSource) {
+  for (const auto &source : m_dataSource) {
     names.push_back(source->getWorkspace()->getName());
   }
   return names;
-- 
GitLab