From 3d76d1afe07957c208036433d0b403d8a8543f64 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?M=C3=A5rten=20Kongstad?= <amhk@google.com>
Date: Tue, 19 Dec 2023 15:21:56 +0100
Subject: [PATCH] aconfig: remove support for --format=bool

This CL is part of a larger effort to simplify maintenance by removing
format aliases: with the new --format option, clients of `aconfig dump`
can specify the format they need themselves.

Bug: 317044914
Test: m
Change-Id: I11efc5d7280402f503ddd05f2dc65bb15ffd8ac7
---
 tools/aconfig/src/dump.rs | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/tools/aconfig/src/dump.rs b/tools/aconfig/src/dump.rs
index bcad064a52..858d14ae91 100644
--- a/tools/aconfig/src/dump.rs
+++ b/tools/aconfig/src/dump.rs
@@ -45,7 +45,6 @@ impl TryFrom<&str> for DumpFormat {
                 "{fully_qualified_name} [{container}]: {permission} + {state} ({trace:paths})"
                     .to_owned(),
             )),
-            "bool" => Ok(Self::Custom("{fully_qualified_name}={state:bool}".to_owned())),
 
             // custom format
             _ => Ok(Self::Custom(value.to_owned())),
@@ -280,10 +279,6 @@ mod tests {
             "verbose",
             "com.android.aconfig.test.enabled_ro [system]: READ_ONLY + ENABLED (tests/test.aconfig, tests/first.values, tests/second.values)"
         );
-        assert_dump_parsed_flags_custom_format_contains!(
-            "bool",
-            "com.android.aconfig.test.enabled_ro=true"
-        );
     }
 
     #[test]