task_manager: do not call result() two times 87/315087/2 accepted/tizen/unified/20240903.110722
authorInki Dae <inki.dae@samsung.com>
Wed, 24 Jul 2024 23:33:10 +0000 (08:33 +0900)
committerInki Dae <inki.dae@samsung.com>
Wed, 24 Jul 2024 23:36:44 +0000 (08:36 +0900)
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>
services/task_manager/src/InferenceNode.cpp

index 3a911b455f9a4628c46a2297a201fa7cbad9d98b..c01d74163721ce4f12e53830412a301e9c6ad655 100644 (file)
@@ -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;
        }