diff --git a/Framework/Algorithms/src/Bin2DPowderDiffraction.cpp b/Framework/Algorithms/src/Bin2DPowderDiffraction.cpp index fdc1bdf12a9401d64b9b95a31bbe32b61faf265c..082a1690e8bc99291dfceeef5945ffee34b85498 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 ff66b18a53145323ea7c58d37ad60b4303781eaa..ca2702cd3eb2adbe225dcd732bbef7e0a987f6d1 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 68f45db705bfb1a5bb50505c9a84246ec6f48b4e..05a165a5428779056fccfb01acc6b0cdf4d75dc5 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 ecfaaba6658a5fa0e8c9fe5ce4fc88ead629934f..ccc851035de87393c7fb8acd8be051aeaef3685f 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 29c367d683f55e2ace73f51f788c2471a2df652c..170e36f77af977e02e5d4d067a7003040d92b110 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 37287a54b107225a0f6175ae7f5d9f2d9382cd07..8d7f330f2986b9c4c1eff9aaf2514950f00a1a7f 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 d619f7672a13db54cdd0089608f635cbd9f8a80f..b6af12842ca5fc78aa4ae7aa0c9b491464a04183 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 2f3e777f21f355eb36ab017a1a793ea154d606b3..8f1398c9ae6a725bd58ab783109e5aad1a9ab5b5 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 ef70edf5def5940ede2d08fcdf03951acc1a1054..8ca44f3a4e1728bfd70a49bd4ccd03af987f9882 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 ¤tRange : reducedRanges) { int nMin = getBestChebyshevPolynomialDegree(dataWorkspace, currentRange); if (nMin > -1) { diff --git a/MantidPlot/src/ConfigDialog.cpp b/MantidPlot/src/ConfigDialog.cpp index 9d4481d99c665381511af311ace31c43b718b5b3..41bcdf182158afb89563c3e1119450ce2bef8fd5 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 45f5424c28f4d60477086750bb86d61c4705d538..718608590cff6e8e22c5cb4de2912f005ac74dd9 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 3e0730a493dd55d5871201ab0e06bb6cadfba77b..2e661c1ca0c62933a29fee488a3858e38fbadeab 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 8621e73dbe89db5d677a348cda163ed23c6c5f6f..23845d0cfeac46470f93bfd9b3263c8db946c63f 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 e799cc191b342abdada73511801b5aff76bdba14..71c4dea80a2159fdca033434e8bf3f4e6739ee54 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 6bdfac3005d7e19cb0ba7afd8582da6aa5698650..4288f92e4185fd52446dddd28e63764960cab3a4 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 932d3599d40b2664e8127b45ca163531d6ac0115..dab707af3b13b415ddc32e721386245fe8baa5b9 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 2383ba7e18aa5ea390aa4fe146260a3cf96bd6d8..c8af0075019289403f11b63e22989ca23876347f 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 fdb2296e86293eafe3b33e0fca384d2b7b0f7f00..47cfb94d1bb054a85470737b280ea29eb211137c 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 5240031851bd26ccd12182e2f9b700cfae664ca3..e3761665e5940d37219dfacf90902dc5519a60e5 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 af4b84c620517fc0173d596a47716d618ba42bd6..8518ad77767fe7b59822dd354be05a8a9e117460 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 d5d371d578026bc942f726601eb911a107da927e..9bc0bf7a5f9cd7f9772e995779c0cd2e84d0b1d5 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 75028395094d0cfe3df01e9df4e0f372e6cfb8ff..9cdf0a13470f930e288ca7d229ca185a2a788ed7 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 ¶meter : 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 917a7fbc8e9fa4087caa1a0cc03db856eec0de9c..645a203a8f1066aaffed964d69c6a5e4ecbe0504 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 2348398c9f3687b0ff66f70eee7f1b1013e22559..c703547ff0b7fa8a98c4bf2a988528b88fb2f079 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 8e13236a4f3c43063c6159924f993a076063ad6b..9221f4d52d23b6fc16205f5cb98532418777b1cb 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 3e47ce81ebbadb6ced35533d4912dfc358846a0c..22d539db6aa555082eb92326fc6c40e446ed2c51 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 §ion : 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 8cbec86f510eaa21cb8b5cdb9619bb03c798cb77..721f8801d61f92742e6cdb6de0b4c9ad5225f5d2 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 §ion : 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 4bd6d1d050c966c2e2eba045b61b5bc5fa09ca74..85a2bc01625a6c87b0a6c676135c44b555945644 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;