diff --git a/source/ArmarXGui/gui-plugins/StatechartEditorPlugin/view/dialogs/RenameStateDialog.cpp b/source/ArmarXGui/gui-plugins/StatechartEditorPlugin/view/dialogs/RenameStateDialog.cpp index 0366eb4a19c946138b55773bb13a4cda3ee630bb..4f0ac580d78a173ca719421cc34c007e801f2bef 100644 --- a/source/ArmarXGui/gui-plugins/StatechartEditorPlugin/view/dialogs/RenameStateDialog.cpp +++ b/source/ArmarXGui/gui-plugins/StatechartEditorPlugin/view/dialogs/RenameStateDialog.cpp @@ -67,13 +67,13 @@ namespace armarx localStates = sourceGroup->getAllStates(true); - for (const StatechartGroupPtr & group : treeModel->getGroups()) + for (const StatechartGroupPtr& group : treeModel->getGroups()) { if (group->getWriteAccess() == StatechartGroup::eReadOnly) { continue; } - for (const StateTreeNodePtr & node : group->getAllNodes()) + for (const StateTreeNodePtr& node : group->getAllNodes()) { if (!node->isState() || !node->getState()) { @@ -82,7 +82,7 @@ namespace armarx auto state = node->getState(); - for (const statechartmodel::StateInstancePtr & instance : state->getSubstates()) + for (const statechartmodel::StateInstancePtr& instance : state->getSubstates()) { if (!instance->getStateClass()) { @@ -120,7 +120,7 @@ namespace armarx int i = 0; - for (const auto & info : instanceRenameInfos) + for (const auto& info : instanceRenameInfos) { QTableWidgetItem* groupItem = new QTableWidgetItem(info.group->getName()); groupItem->setFlags(groupItem->flags() ^ Qt::ItemIsEditable); @@ -250,7 +250,7 @@ namespace armarx { bool inUse = false; - for (const auto & s : localStates) + for (const auto& s : localStates) { if (newName == s->getStateName()) { @@ -272,7 +272,7 @@ namespace armarx colorToUse = colorGreen; } - validNewName = !inUse && !newName.size() == 0; + validNewName = (!inUse && newName.size() != 0); setOkButtonsEnabled(validNewName && validInstanceNames); QPalette p(ui->labelNewNameError->palette());