commit 3dfb8bc8892e1e0d09e216751b93cc1cc1fb941c
author Bart Van Assche <bvanassche@google.com> Wed Feb 22 10:01:33 2023 -0800
committer Bart Van Assche <bvanassche@google.com> Thu Feb 23 16:06:42 2023 +0000
tree f9821f4e60e94f44b9d0785fb0cbf9d8b94a16c9
parent bae352bddbf47e7bd97f415e510dab34ba97d6ae

init: Simplify struct BuiltinArguments

Make the code that creates BuiltinArguments instances easier to read by
using initializer lists instead of constructor calls. Remove the
BuiltinArguments constructors.

Change-Id: I6cf215a81d298cf7e524e22fb75db820e0225c9a
Signed-off-by: Bart Van Assche <bvanassche@google.com>

init/action.cpp

diff --git a/init/action.cpp b/init/action.cpp
index 1e998ae..18f6360 100644
--- a/init/action.cpp
+++ b/init/action.cpp
@@ -30,7 +30,7 @@
 
 Result<void> RunBuiltinFunction(const BuiltinFunction& function,
                                 const std::vector<std::string>& args, const std::string& context) {
-    auto builtin_arguments = BuiltinArguments(context);
+    BuiltinArguments builtin_arguments{.context = context};
 
     builtin_arguments.args.resize(args.size());
     builtin_arguments.args[0] = args[0];
@@ -69,7 +69,7 @@
 }
 
 Result<void> Command::CheckCommand() const {
-    auto builtin_arguments = BuiltinArguments("host_init_verifier");
+    BuiltinArguments builtin_arguments{.context = "host_init_verifier"};
 
     builtin_arguments.args.resize(args_.size());
     builtin_arguments.args[0] = args_[0];