From: Hosang Kim <hosang12.kim@samsung.com>
Date: Wed, 30 Apr 2025 06:34:15 +0000 (+0900)
Subject: fix coverity
X-Git-Tag: accepted/tizen/unified/20250509.120514~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f878b3f1aed317faac08bc8e7b3d7fe9e8078012;p=platform%2Fcore%2Fuifw%2Faurum.git

fix coverity

- COPY INSTEAD OF MOVE

Change-Id: I916b7bebcf54045e6bce34cb68f15f083f6b02e4
---

diff --git a/libaurum/inc/Until.h b/libaurum/inc/Until.h
index 0fd797e..f1f1abf 100644
--- a/libaurum/inc/Until.h
+++ b/libaurum/inc/Until.h
@@ -88,7 +88,7 @@ public:
      * @since_tizen 6.5
      */
     static std::function<bool(const ISearchable *)> hasObject(
-        const std::shared_ptr<UiSelector> selector);
+        const std::shared_ptr<UiSelector> &selector);
 
     /**
      * @brief Checks that there is an object that are satisfied with selector condition.
@@ -101,7 +101,7 @@ public:
      * @since_tizen 6.5
      */
     static std::function<std::shared_ptr<UiObject>(const ISearchable *)>
-                                                 findObject(const std::shared_ptr<UiSelector> selector);
+                                                 findObject(const std::shared_ptr<UiSelector> &selector);
 
     /**
      * @brief Checks the given object on checkable state.
@@ -127,7 +127,7 @@ public:
      * @since_tizen 7.0
      */
     static std::function<std::vector<std::shared_ptr<UiObject>>(const ISearchable *)>
-                                                 findObjects(const std::shared_ptr<UiSelector> selector);
+                                                 findObjects(const std::shared_ptr<UiSelector> &selector);
 };
 
 }
diff --git a/libaurum/src/Until.cc b/libaurum/src/Until.cc
index 0affb12..0843f92 100644
--- a/libaurum/src/Until.cc
+++ b/libaurum/src/Until.cc
@@ -20,7 +20,7 @@
 using namespace Aurum;
 
 std::function<bool(const ISearchable *)> Until::hasObject(
-    const std::shared_ptr<UiSelector> selector)
+    const std::shared_ptr<UiSelector> &selector)
 {
     return [=](const ISearchable *searchable) -> bool {
         LOGI("hasObject search:%p", searchable);
@@ -30,7 +30,7 @@ std::function<bool(const ISearchable *)> Until::hasObject(
 }
 
 std::function<std::shared_ptr<UiObject>(const ISearchable *)> Until::findObject(
-    const std::shared_ptr<UiSelector> selector)
+    const std::shared_ptr<UiSelector> &selector)
 {
     return [=](const ISearchable *searchable) -> std::shared_ptr<UiObject> {
         LOGI("findObject search:%p", searchable);
@@ -48,7 +48,7 @@ std::function<bool(const UiObject *)> Until::checkable(const bool isCheckable)
 }
 
 std::function<std::vector<std::shared_ptr<UiObject>>(const ISearchable *)> Until::findObjects(
-    const std::shared_ptr<UiSelector> selector)
+    const std::shared_ptr<UiSelector> &selector)
 {
     return [=](const ISearchable *searchable) -> std::vector<std::shared_ptr<UiObject>> {
         LOGI("findObjects search:%p", searchable);
diff --git a/org.tizen.aurum-bootstrap/inc/Commands/DumpObjectTreeCommand.h b/org.tizen.aurum-bootstrap/inc/Commands/DumpObjectTreeCommand.h
index f248436..56aff84 100644
--- a/org.tizen.aurum-bootstrap/inc/Commands/DumpObjectTreeCommand.h
+++ b/org.tizen.aurum-bootstrap/inc/Commands/DumpObjectTreeCommand.h
@@ -31,5 +31,5 @@ public:
     ::grpc::Status execute() override;
 
 private:
-    void traverse(::aurum::Element *root, std::shared_ptr<Node> node, int depth);
+    void traverse(::aurum::Element *root, const std::shared_ptr<Node> &node, int depth);
 };
diff --git a/org.tizen.aurum-bootstrap/src/Commands/DumpObjectTreeCommand.cc b/org.tizen.aurum-bootstrap/src/Commands/DumpObjectTreeCommand.cc
index a98bfe4..91e7837 100644
--- a/org.tizen.aurum-bootstrap/src/Commands/DumpObjectTreeCommand.cc
+++ b/org.tizen.aurum-bootstrap/src/Commands/DumpObjectTreeCommand.cc
@@ -30,7 +30,7 @@ DumpObjectTreeCommand::DumpObjectTreeCommand(const ::aurum::ReqDumpObjectTree *r
     mObjMap = ObjectMapper::getInstance();
 }
 
-void DumpObjectTreeCommand::traverse(::aurum::Element *root, std::shared_ptr<Node> node, int depth)
+void DumpObjectTreeCommand::traverse(::aurum::Element *root, const std::shared_ptr<Node> &node, int depth)
 {
     if (!node->mNode) return;
     std::shared_ptr<UiObject> obj = node->mNode;