From 701589b6510f38393feda36ddbbb3e0b6f4a71d5 Mon Sep 17 00:00:00 2001 From: v4hn Date: Fri, 24 Apr 2020 16:57:38 +0200 Subject: [PATCH] simplify reading my eyes hurt trying to cope with it.first->second . --- visualization/motion_planning_tasks/src/task_list_model.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/visualization/motion_planning_tasks/src/task_list_model.cpp b/visualization/motion_planning_tasks/src/task_list_model.cpp index 31a57a3d..0f76e6f3 100644 --- a/visualization/motion_planning_tasks/src/task_list_model.cpp +++ b/visualization/motion_planning_tasks/src/task_list_model.cpp @@ -251,7 +251,8 @@ void TaskListModel::processTaskDescriptionMessage(const std::string& id, // retrieve existing or insert new remote task for given id auto it_inserted = remote_tasks_.insert(std::make_pair(id, nullptr)); bool created = it_inserted.second; - RemoteTaskModel*& remote_task = it_inserted.first->second; + const auto& task_it = it_inserted.first; + RemoteTaskModel*& remote_task = task_it->second; if (!msg.stages.empty() && remote_task && (remote_task->taskFlags() & BaseTaskModel::IS_DESTROYED)) { removeModel(remote_task); @@ -262,7 +263,7 @@ void TaskListModel::processTaskDescriptionMessage(const std::string& id, if (msg.stages.empty()) { if (!remote_task) { // task was already deleted locally // we can now remove it from remote_tasks_ - remote_tasks_.erase(it_inserted.first); + remote_tasks_.erase(task_it); return; } } else if (created) { // create new task model, if ID was not known before