diff --git a/Framework/API/test/MatrixWorkspaceTest.h b/Framework/API/test/MatrixWorkspaceTest.h index fe51bb9a5f104597177efbab9b013b4ac7eb24ed..d925f6f91b9a3d0bbbcc330422b195114b048aab 100644 --- a/Framework/API/test/MatrixWorkspaceTest.h +++ b/Framework/API/test/MatrixWorkspaceTest.h @@ -1611,7 +1611,6 @@ public: TS_ASSERT(l2 > 0); } - /* * Rotate a bank in the workspace and read the positions out again. Very * typical. diff --git a/Framework/Algorithms/src/ConvertAxisByFormula.cpp b/Framework/Algorithms/src/ConvertAxisByFormula.cpp index 050fb69fc574bca7cdf9dffe1b80f842e0cbe004..ad2dbbf1b1de0d63b134f151e425b95eefbfe377 100644 --- a/Framework/Algorithms/src/ConvertAxisByFormula.cpp +++ b/Framework/Algorithms/src/ConvertAxisByFormula.cpp @@ -277,9 +277,10 @@ void ConvertAxisByFormula::calculateValues( } } -void ConvertAxisByFormula::setGeometryValues( - const API::SpectrumInfo &specInfo, const size_t index, - std::vector<Variable_ptr> &variables) { +void +ConvertAxisByFormula::setGeometryValues(const API::SpectrumInfo &specInfo, + const size_t index, + std::vector<Variable_ptr> &variables) { for (auto variable : variables) { if (variable->isGeometric) { if (variable->name == "twotheta") { diff --git a/Framework/Algorithms/src/MergeRuns/SampleLogsBehaviour.cpp b/Framework/Algorithms/src/MergeRuns/SampleLogsBehaviour.cpp index 000c88e698ea62aaad1286325eec265e008f3209..89456f0c700f97aa0115157835a2504ca3023647 100644 --- a/Framework/Algorithms/src/MergeRuns/SampleLogsBehaviour.cpp +++ b/Framework/Algorithms/src/MergeRuns/SampleLogsBehaviour.cpp @@ -500,8 +500,9 @@ bool SampleLogsBehaviour::isWithinTolerance(const SampleLogBehaviour &behaviour, * @param addeeWSProperty a pointer to the property in the workspace being added * @return true if the sample logs match, false otherwise */ -bool SampleLogsBehaviour::stringPropertiesMatch( - const SampleLogBehaviour &behaviour, const Property *addeeWSProperty) { +bool +SampleLogsBehaviour::stringPropertiesMatch(const SampleLogBehaviour &behaviour, + const Property *addeeWSProperty) { return behaviour.property->value().compare(addeeWSProperty->value()) == 0; } diff --git a/Framework/CurveFitting/src/HistogramDomainCreator.cpp b/Framework/CurveFitting/src/HistogramDomainCreator.cpp index 59f68c4fae485b7db188cc15d25d1ab69c3a2f36..593b87046a9f7dc136b2a96f7b0f131f9d408fda 100644 --- a/Framework/CurveFitting/src/HistogramDomainCreator.cpp +++ b/Framework/CurveFitting/src/HistogramDomainCreator.cpp @@ -29,9 +29,10 @@ HistogramDomainCreator::HistogramDomainCreator( * @param values :: Pointer to outgoing FunctionValues object. * @param i0 :: Size offset for values object if it already contains data. */ -void HistogramDomainCreator::createDomain( - boost::shared_ptr<FunctionDomain> &domain, - boost::shared_ptr<FunctionValues> &values, size_t i0) { +void +HistogramDomainCreator::createDomain(boost::shared_ptr<FunctionDomain> &domain, + boost::shared_ptr<FunctionValues> &values, + size_t i0) { setParameters(); diff --git a/Framework/DataHandling/src/LoadMask.cpp b/Framework/DataHandling/src/LoadMask.cpp index 6544f336eb354bac7f9e8469a64d742aed68252d..56a35c341220f35d055700665c4b86d9f3e15750 100644 --- a/Framework/DataHandling/src/LoadMask.cpp +++ b/Framework/DataHandling/src/LoadMask.cpp @@ -403,9 +403,9 @@ void LoadMask::initDetectors() { * @param tomask: true to mask, false to unmask * @param singledetids: list of individual det ids to mask */ -void LoadMask::processMaskOnDetectors( - const detid2index_map &indexmap, bool tomask, - const std::vector<detid_t> &singledetids) { +void +LoadMask::processMaskOnDetectors(const detid2index_map &indexmap, bool tomask, + const std::vector<detid_t> &singledetids) { // 1. Get index map // 2. Mask g_log.debug() << "Mask = " << tomask @@ -434,9 +434,9 @@ void LoadMask::processMaskOnDetectors( * @param detectors -- vector of detector ids, which belongs to components *provided as input. */ -void LoadMask::componentToDetectors( - const std::vector<std::string> &componentnames, - std::vector<detid_t> &detectors) { +void +LoadMask::componentToDetectors(const std::vector<std::string> &componentnames, + std::vector<detid_t> &detectors) { Geometry::Instrument_const_sptr minstrument = m_maskWS->getInstrument(); for (auto &componentname : componentnames) { diff --git a/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorGenerateNotebook.h b/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorGenerateNotebook.h index 90e576aacb08f6803773f7c5a65d200fdf43a3d5..714115c93b946f3cf5dd362f296cfaff2dbae9ae 100644 --- a/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorGenerateNotebook.h +++ b/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorGenerateNotebook.h @@ -52,8 +52,8 @@ splitByCommas(const std::string &names_string); std::string DLLExport plot1DString(const std::vector<std::string> &ws_names); -std::string DLLExport -tableString(const TreeData &treeData, const DataProcessorWhiteList &whitelist); +std::string DLLExport tableString(const TreeData &treeData, + const DataProcessorWhiteList &whitelist); std::string DLLExport titleString(const std::string &wsName); @@ -69,9 +69,9 @@ plotsString(const std::vector<std::string> &output_ws, const DataProcessorProcessingAlgorithm &processor); std::string DLLExport -getReducedWorkspaceName(const RowData &data, - const DataProcessorWhiteList &whitelist, - const std::string &prefix = ""); + getReducedWorkspaceName(const RowData &data, + const DataProcessorWhiteList &whitelist, + const std::string &prefix = ""); boost::tuple<std::string, std::string> DLLExport reduceRowString( const RowData &data, const std::string &instrument, diff --git a/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorOpenTableCommand.h b/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorOpenTableCommand.h index c4e030835479659da613410424cf99a3c707ea0e..3c69d99c29c2e851e223e9159785e62b273998bc 100644 --- a/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorOpenTableCommand.h +++ b/MantidQt/MantidWidgets/inc/MantidQtMantidWidgets/DataProcessorUI/DataProcessorOpenTableCommand.h @@ -36,8 +36,7 @@ public: : DataProcessorCommandBase(tablePresenter){}; virtual ~DataProcessorOpenTableCommand(){}; - void execute() override{ - // This action should do nothing + void execute() override{// This action should do nothing }; std::string name() override { return std::string("Open Table"); } std::string icon() override { return std::string("://multiload.png"); } diff --git a/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorOneLevelTreeManager.cpp b/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorOneLevelTreeManager.cpp index c8a059e20d30de80a26a0cac200cacc0130a63e4..a4522133c07d6a172fec3ba84179ecbde2cb43d1 100644 --- a/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorOneLevelTreeManager.cpp +++ b/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorOneLevelTreeManager.cpp @@ -356,8 +356,9 @@ void DataProcessorOneLevelTreeManager::transfer( * @param child :: the row * @param data :: the data */ -void DataProcessorOneLevelTreeManager::update( - int parent, int child, const std::vector<std::string> &data) { +void +DataProcessorOneLevelTreeManager::update(int parent, int child, + const std::vector<std::string> &data) { UNUSED_ARG(child); @@ -407,8 +408,9 @@ ITableWorkspace_sptr DataProcessorOneLevelTreeManager::createDefaultWorkspace( * @param ws :: the table workspace * @param whitelistColumns :: the number of columns as specified in a whitelist */ -void DataProcessorOneLevelTreeManager::validateModel( - ITableWorkspace_sptr ws, size_t whitelistColumns) const { +void +DataProcessorOneLevelTreeManager::validateModel(ITableWorkspace_sptr ws, + size_t whitelistColumns) const { if (!ws) throw std::runtime_error("Null pointer"); @@ -435,8 +437,9 @@ void DataProcessorOneLevelTreeManager::validateModel( * @param whitelistColumns : [input] The number of columns in the whitelist * @throws std::runtime_error if the number of columns in the table is incorrect */ -bool DataProcessorOneLevelTreeManager::isValidModel( - Workspace_sptr ws, size_t whitelistColumns) const { +bool +DataProcessorOneLevelTreeManager::isValidModel(Workspace_sptr ws, + size_t whitelistColumns) const { try { validateModel(boost::dynamic_pointer_cast<ITableWorkspace>(ws), diff --git a/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorTwoLevelTreeManager.cpp b/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorTwoLevelTreeManager.cpp index 250a97c49295091c612e6ef33aa46cd60a19c0f4..50c6f83577a0ff9d6ee3999c4e86eb6f1b1a10b0 100644 --- a/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorTwoLevelTreeManager.cpp +++ b/MantidQt/MantidWidgets/src/DataProcessorUI/DataProcessorTwoLevelTreeManager.cpp @@ -551,8 +551,9 @@ void DataProcessorTwoLevelTreeManager::transfer( * @param child :: the row * @param data :: the data */ -void DataProcessorTwoLevelTreeManager::update( - int parent, int child, const std::vector<std::string> &data) { +void +DataProcessorTwoLevelTreeManager::update(int parent, int child, + const std::vector<std::string> &data) { if (static_cast<int>(data.size()) != m_model->columnCount()) throw std::invalid_argument("Can't update tree with given data"); @@ -605,8 +606,9 @@ ITableWorkspace_sptr DataProcessorTwoLevelTreeManager::createDefaultWorkspace( * @param ws :: the table workspace * @param whitelistColumns :: the number of columns as specified in a whitelist */ -void DataProcessorTwoLevelTreeManager::validateModel( - ITableWorkspace_sptr ws, size_t whitelistColumns) const { +void +DataProcessorTwoLevelTreeManager::validateModel(ITableWorkspace_sptr ws, + size_t whitelistColumns) const { if (!ws) throw std::runtime_error("Null pointer"); @@ -633,8 +635,9 @@ void DataProcessorTwoLevelTreeManager::validateModel( * @param whitelistColumns : [input] The number of columns in the whitelist * @throws std::runtime_error if the number of columns in the table is incorrect */ -bool DataProcessorTwoLevelTreeManager::isValidModel( - Workspace_sptr ws, size_t whitelistColumns) const { +bool +DataProcessorTwoLevelTreeManager::isValidModel(Workspace_sptr ws, + size_t whitelistColumns) const { try { validateModel(boost::dynamic_pointer_cast<ITableWorkspace>(ws), diff --git a/MantidQt/MantidWidgets/src/DataProcessorUI/GenericDataProcessorPresenter.cpp b/MantidQt/MantidWidgets/src/DataProcessorUI/GenericDataProcessorPresenter.cpp index 7f7d160e42b410f7b0828a266541be545a01effe..938372263f2af7d37db46ffaa2690fbb2e84c2c0 100644 --- a/MantidQt/MantidWidgets/src/DataProcessorUI/GenericDataProcessorPresenter.cpp +++ b/MantidQt/MantidWidgets/src/DataProcessorUI/GenericDataProcessorPresenter.cpp @@ -275,8 +275,8 @@ void GenericDataProcessorPresenter::saveNotebook(const TreeData &data) { Post-processes the workspaces created by the given rows together. @param groupData : the data in a given group as received from the tree manager */ -void GenericDataProcessorPresenter::postProcessGroup( - const GroupData &groupData) { +void +GenericDataProcessorPresenter::postProcessGroup(const GroupData &groupData) { if (!m_postprocess) throw std::runtime_error("Cannot post-process workspaces"); @@ -1245,8 +1245,9 @@ ChildItems GenericDataProcessorPresenter::selectedChildren() const { * @param title :: the title * @return :: Yes/No */ -bool GenericDataProcessorPresenter::askUserYesNo( - const std::string &prompt, const std::string &title) const { +bool +GenericDataProcessorPresenter::askUserYesNo(const std::string &prompt, + const std::string &title) const { return m_mainPresenter->askUserYesNo(prompt, title); } @@ -1255,8 +1256,9 @@ bool GenericDataProcessorPresenter::askUserYesNo( * @param prompt :: the message * @param title :: the title */ -void GenericDataProcessorPresenter::giveUserWarning( - const std::string &prompt, const std::string &title) const { +void +GenericDataProcessorPresenter::giveUserWarning(const std::string &prompt, + const std::string &title) const { m_mainPresenter->giveUserWarning(prompt, title); } diff --git a/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorTwoLevelTreeModel.cpp b/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorTwoLevelTreeModel.cpp index 7dfec86fb232e265cc62b7b4f4c323ad1640b4cd..88e4c14e0b0a044b622e2901f55bde226d304e22 100644 --- a/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorTwoLevelTreeModel.cpp +++ b/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorTwoLevelTreeModel.cpp @@ -425,8 +425,8 @@ bool QDataProcessorTwoLevelTreeModel::setData(const QModelIndex &index, * whitelist * @param table : A table workspace containing the data */ -void QDataProcessorTwoLevelTreeModel::setupModelData( - ITableWorkspace_sptr table) { +void +QDataProcessorTwoLevelTreeModel::setupModelData(ITableWorkspace_sptr table) { int nrows = static_cast<int>(table->rowCount()); diff --git a/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorWidget.cpp b/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorWidget.cpp index fb8b8b26e0fc952385c5240c4e657be233883c4d..4c41b5fdc71aa5f7af861cf643823db6494ca6ee 100644 --- a/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorWidget.cpp +++ b/MantidQt/MantidWidgets/src/DataProcessorUI/QDataProcessorWidget.cpp @@ -109,8 +109,8 @@ void QDataProcessorWidget::setModel(const std::string &name) { Set a new model in the tableview @param model : the model to be attached to the tableview */ -void QDataProcessorWidget::showTable( - boost::shared_ptr<QAbstractItemModel> model) { +void +QDataProcessorWidget::showTable(boost::shared_ptr<QAbstractItemModel> model) { m_model = model; // So we can notify the presenter when the user updates the table connect(m_model.get(), diff --git a/MantidQt/SliceViewer/src/ProxyCompositePeaksPresenter.cpp b/MantidQt/SliceViewer/src/ProxyCompositePeaksPresenter.cpp index db3b4af0b72bf463f9fc9bda2dd8e6f4007c71c2..0b0b67284fa95fffa63ef8dd11111c84a9e3f801 100644 --- a/MantidQt/SliceViewer/src/ProxyCompositePeaksPresenter.cpp +++ b/MantidQt/SliceViewer/src/ProxyCompositePeaksPresenter.cpp @@ -161,13 +161,13 @@ bool ProxyCompositePeaksPresenter::hasPeakAddModeFor( return m_compositePresenter->hasPeakAddModeFor(target); } -void ProxyCompositePeaksPresenter::setPeakSizeOnProjection( - const double fraction) { +void +ProxyCompositePeaksPresenter::setPeakSizeOnProjection(const double fraction) { m_compositePresenter->setPeakSizeOnProjection(fraction); } -void ProxyCompositePeaksPresenter::setPeakSizeIntoProjection( - const double fraction) { +void +ProxyCompositePeaksPresenter::setPeakSizeIntoProjection(const double fraction) { m_compositePresenter->setPeakSizeIntoProjection(fraction); }