From cebcd85b687f2515ebbc08a3412d65db9eb42926 Mon Sep 17 00:00:00 2001 From: Inki Dae Date: Thu, 25 Jul 2024 08:33:10 +0900 Subject: [PATCH] task_manager: do not call result() two times Do not call result() two times. We can use a reference instead of calling result() again. Change-Id: I0378f9a533cbacd146b57168aa60e17d5947cfca Signed-off-by: Inki Dae --- services/task_manager/src/InferenceNode.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/services/task_manager/src/InferenceNode.cpp b/services/task_manager/src/InferenceNode.cpp index 3a911b4..c01d741 100644 --- a/services/task_manager/src/InferenceNode.cpp +++ b/services/task_manager/src/InferenceNode.cpp @@ -59,9 +59,10 @@ void InferenceNode::invoke() _task->invoke(*inputs[0]); _status = NodeStatus::INVALID; - if (!_task->result()._is_empty) { + auto &result = _task->result(); + if (!result._is_empty) { _resultMutex.lock(); - _results.push_back(_task->result().clone()); + _results.push_back(result.clone()); _resultMutex.unlock(); _status = NodeStatus::VALID; } -- 2.34.1