From: Inki Dae <inki.dae@samsung.com>
Date: Wed, 24 Jul 2024 23:33:10 +0000 (+0900)
Subject: task_manager: do not call result() two times
X-Git-Tag: accepted/tizen/unified/20240903.110722^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cebcd85b687f2515ebbc08a3412d65db9eb42926;p=platform%2Fcore%2Fapi%2Fsingleo.git

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 <inki.dae@samsung.com>
---

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;
 	}