Skip to content

Commit 8d6c354

Browse files
committed
Make AndroidEfiLoaderFlag inherit from FlagBase
Bug: b/472361611
1 parent 72c13cd commit 8d6c354

File tree

4 files changed

+6
-18
lines changed

4 files changed

+6
-18
lines changed

base/cvd/cuttlefish/host/commands/assemble_cvd/disk_image_flags_vectorization.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ Result<void> DiskImageFlagsVectorization(
139139
}
140140
instance.set_super_image(super_image.SuperImageForIndex(instance_index));
141141
instance.set_android_efi_loader(
142-
android_efi_loader.AndroidEfiLoaderForInstance(instance_index));
142+
android_efi_loader.ForIndex(instance_index));
143143
if (instance_index >= chromeos_disk.size()) {
144144
instance.set_chromeos_disk(chromeos_disk[0]);
145145
} else {

base/cvd/cuttlefish/host/commands/assemble_cvd/flags/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ cf_cc_library(
2525
deps = [
2626
"//cuttlefish/common/libs/utils:files",
2727
"//cuttlefish/host/commands/assemble_cvd:flags_defaults",
28+
"//cuttlefish/host/commands/assemble_cvd/flags:flag_base",
2829
"//cuttlefish/host/commands/assemble_cvd/flags:system_image_dir",
2930
"//cuttlefish/host/commands/assemble_cvd/flags:vm_manager",
3031
"//cuttlefish/host/libs/config:vmm_mode",

base/cvd/cuttlefish/host/commands/assemble_cvd/flags/android_efi_loader.cc

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -69,17 +69,6 @@ AndroidEfiLoaderFlag AndroidEfiLoaderFlag::FromGlobalGflags(
6969
}
7070

7171
AndroidEfiLoaderFlag::AndroidEfiLoaderFlag(std::vector<std::string> paths)
72-
: paths_(std::move(paths)) {}
73-
74-
std::string AndroidEfiLoaderFlag::AndroidEfiLoaderForInstance(
75-
size_t instance_index) const {
76-
if (instance_index < paths_.size()) {
77-
return paths_[instance_index];
78-
} else if (!paths_.empty()) {
79-
return paths_[0];
80-
} else {
81-
return "";
82-
}
83-
}
72+
: FlagBase(paths) {}
8473

8574
} // namespace cuttlefish

base/cvd/cuttlefish/host/commands/assemble_cvd/flags/android_efi_loader.h

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,23 +20,21 @@
2020
#include <string>
2121
#include <vector>
2222

23+
#include "cuttlefish/host/commands/assemble_cvd/flags/flag_base.h"
2324
#include "cuttlefish/host/commands/assemble_cvd/flags/system_image_dir.h"
2425
#include "cuttlefish/host/commands/assemble_cvd/flags/vm_manager.h"
2526

2627
namespace cuttlefish {
2728

2829
/* `--android_efi_loader` flag */
29-
class AndroidEfiLoaderFlag {
30+
class AndroidEfiLoaderFlag : public FlagBase<std::string> {
3031
public:
3132
static AndroidEfiLoaderFlag FromGlobalGflags(const SystemImageDirFlag&,
3233
const VmManagerFlag&);
33-
34-
std::string AndroidEfiLoaderForInstance(size_t instance_index) const;
34+
~AndroidEfiLoaderFlag() override = default;
3535

3636
private:
3737
AndroidEfiLoaderFlag(std::vector<std::string>);
38-
39-
std::vector<std::string> paths_;
4038
};
4139

4240
} // namespace cuttlefish

0 commit comments

Comments
 (0)