StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

aconfig: remove the 'debug' dump format

Browse Source 
The --format=debug dump output format is largely superseded by the
textproto format now that aconfig doesn't use hand-written wrappers
around the auto-generated proto struct anymore. Remove the format to
reduce the risk of code rot.

Bug: 283910447
Test: atest aconfig.test
Change-Id: I6700fc4eafd3fa3a63952109c0105d34c7ffd98b
This commit is contained in:
Mårten Kongstad
2023-07-20 09:25:56 +02:00
parent 9c3502d4b4
commit 9338007cff
1 changed files with 0 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tools/aconfig/src/commands.rs
View File

@@ -213,7 +213,6 @@ pub fn create_device_config_sysprops(mut input: Input) -> Result<Vec<u8>> {
#[derive(Copy, Clone, Debug, PartialEq, Eq, ValueEnum)] #[derive(Copy, Clone, Debug, PartialEq, Eq, ValueEnum)]
pub enum DumpFormat { pub enum DumpFormat {
Text, Text,
Debug,
Protobuf, Protobuf,
Textproto, Textproto,
} }
@@ -238,12 +237,6 @@ pub fn dump_parsed_flags(mut input: Vec<Input>, format: DumpFormat) -> Result<Ve
output.extend_from_slice(line.as_bytes()); output.extend_from_slice(line.as_bytes());
} }
} }
DumpFormat::Debug => {
for parsed_flag in parsed_flags.parsed_flag.into_iter() {
let line = format!("{:#?}\n", parsed_flag);
output.extend_from_slice(line.as_bytes());
}
}
DumpFormat::Protobuf => { DumpFormat::Protobuf => {
parsed_flags.write_to_vec(&mut output)?; parsed_flags.write_to_vec(&mut output)?;
} }