commit 431f44a31481014e339086a34c7369813d1fb8a7
author Ted Bauer <tedbauer@google.com> Fri Sep 13 20:59:34 2024 +0000
committer Ted Bauer <tedbauer@google.com> Mon Sep 16 15:50:50 2024 +0000
tree 8f1fa39e145620f6721c5e62dce155e01da8eec5
parent 5d7cbb65792a7321a9656a31e8b505ec15e09be5

Don't show staged value if flag is RO

Bug: 324436145
Change-Id: I493421f29d6e7f5e844dd47cc6417b3d5fe9cdbb
Test: m aflags && aflags list

tools/aconfig/aflags/src/main.rs

diff --git a/tools/aconfig/aflags/src/main.rs b/tools/aconfig/aflags/src/main.rs
index a726cc0..07b7243 100644
--- a/tools/aconfig/aflags/src/main.rs
+++ b/tools/aconfig/aflags/src/main.rs
@@ -116,9 +116,10 @@
     }
 
     fn display_staged_value(&self) -> String {
-        match self.staged_value {
-            Some(v) => format!("(->{})", v),
-            None => "-".to_string(),
+        match (&self.permission, self.staged_value) {
+            (FlagPermission::ReadOnly, _) => "-".to_string(),
+            (FlagPermission::ReadWrite, None) => "-".to_string(),
+            (FlagPermission::ReadWrite, Some(v)) => format!("(->{})", v),
         }
     }
 }