sensord: use emplace_back instead of push_back

Change-Id: I812a34b162a982dfa1ba7033bcb35e758d8d71dd
Signed-off-by: kibak.yoon <seseki17@gmail.com>

diff --git a/src/sensorctl/test_bench.cpp b/src/sensorctl/test_bench.cpp
index 201a7f5..3b9cd51 100644 (file)
--- a/src/sensorctl/test_bench.cpp
+++ b/src/sensorctl/test_bench.cpp
@@ -133,9 +133,7 @@ void test_bench::add_testcase(const std::string &group, test_case *testcase)
 
 void test_bench::add_failure(const std::string &function, long line, const std::string &msg)
 {
-       test_result fail(function, line, msg);
-
-       results.push_back(fail);
+       results.emplace_back(function, line, msg);
        m_failure_count++;
 }
 

diff --git a/src/server/sensor_loader.cpp b/src/server/sensor_loader.cpp
index 4d1d20e..9ea7e7f 100644 (file)
--- a/src/server/sensor_loader.cpp
+++ b/src/server/sensor_loader.cpp
@@ -73,7 +73,6 @@ bool sensor_loader::load(const std::string &dir_path, std::vector<std::shared_pt
        void *handle;
        std::vector<std::string> module_paths;
        void **results;
-       T *sensor;
 
        ret = get_module_paths(dir_path, module_paths);
        retv_if(!ret, false);
@@ -97,11 +96,8 @@ bool sensor_loader::load(const std::string &dir_path, std::vector<std::shared_pt
                        return false;
                }
 
-               for (int i = 0; i < size; ++i) {
-                       sensor = static_cast<T *>(results[i]);
-                       std::shared_ptr<T> sensor_ptr(sensor);
-                       sensors.push_back(sensor_ptr);
-               }
+               for (int i = 0; i < size; ++i)
+                       sensors.emplace_back(static_cast<T *>(results[i]));
 
                m_modules[path.c_str()] = handle;
        }