commit 437f0b27d51fbfdeb65f2d4ec8f0dd26bdbfbc37
author Pavithra Barithaya <pavithrabarithaya07@gmail.com> Fri Jun 21 07:58:37 2024 -0500
committer Pavithra Barithaya <pavithrabarithaya07@gmail.com> Fri Jun 21 09:12:09 2024 -0500
tree bac6905599eaded860a09ea229ed236ee35a57a2
parent 66a93fc80ffda90db38065ce0ea630d07f28ad01

clang-tidy: Enable modernize-use-emplace check

The check flags insertions to an STL-style container done by
calling the push_back, push, or push_front methods with an
explicitly-constructed temporary of the container element type.
In this case, the corresponding emplace equivalent methods result
in less verbose and potentially more efficient code.

Change-Id: I1e7ae19ef1400c83717b2df48f3314ba4e96423e
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>

test/systemd_parser.cpp

diff --git a/test/systemd_parser.cpp b/test/systemd_parser.cpp
index 7277f60..65fbec4 100644
--- a/test/systemd_parser.cpp
+++ b/test/systemd_parser.cpp
@@ -49,8 +49,8 @@
     std::fclose(tmpf);
 
     std::vector<std::string> filePaths;
-    filePaths.push_back("/tmp/good_file1.json");
-    filePaths.push_back("/tmp/good_file2.json");
+    filePaths.emplace_back("/tmp/good_file1.json");
+    filePaths.emplace_back("/tmp/good_file2.json");
 
     TargetErrorData targetData = parseFiles(filePaths);
 
@@ -89,7 +89,7 @@
     std::fclose(tmpf);
 
     std::vector<std::string> filePaths;
-    filePaths.push_back("/tmp/invalid_error_file.json");
+    filePaths.emplace_back("/tmp/invalid_error_file.json");
 
     // Verify exception thrown on invalid errorsToMonitor
     EXPECT_THROW(TargetErrorData targetData = parseFiles(filePaths),
@@ -104,7 +104,7 @@
     fclose(tmpf);
 
     std::vector<std::string> filePaths;
-    filePaths.push_back("/tmp/invalid_json_file.json");
+    filePaths.emplace_back("/tmp/invalid_json_file.json");
 
     // Verify exception thrown on invalid json file format
     EXPECT_THROW(TargetErrorData targetData = parseFiles(filePaths),
@@ -129,7 +129,7 @@
     std::fclose(tmpf);
 
     std::vector<std::string> filePaths;
-    filePaths.push_back("/tmp/not_just_default_file.json");
+    filePaths.emplace_back("/tmp/not_just_default_file.json");
 
     // Verify exception thrown on invalid errorsToMonitor
     EXPECT_THROW(TargetErrorData targetData = parseFiles(filePaths),