commit f9ec45d890c2a4ff21151a72e9fe023ac1d03594
author Martin Geisler <mgeisler@google.com> Tue May 02 09:07:30 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Tue May 02 09:07:30 2023 +0000
tree ffb08e2218a96bba0e04790ae747f0bd9f869f1b
parent 77a3c23e4dc49872a82add88073ad8a6c2d352b3
parent 631fdff301460ccd9e78810c41313a2a43a48c59

Simplify bool parsing am: 8aa7e9be3e am: 631fdff301

Original change: https://android-review.googlesource.com/c/platform/system/librustutils/+/2571571

Change-Id: I35eaa2f478d4e0d8fbefbfecfead2146f3c06c3e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

system_properties.rs

diff --git a/system_properties.rs b/system_properties.rs
index 9db52bc..f2ceeeb 100644
--- a/system_properties.rs
+++ b/system_properties.rs
@@ -225,12 +225,10 @@
 }
 
 fn parse_bool(value: &str) -> Option<bool> {
-    if ["1", "y", "yes", "on", "true"].contains(&value) {
-        Some(true)
-    } else if ["0", "n", "no", "off", "false"].contains(&value) {
-        Some(false)
-    } else {
-        None
+    match value {
+        "1" | "y" | "yes" | "on" | "true" => Some(true),
+        "0" | "n" | "no" | "off" | "false" => Some(false),
+        _ => None,
     }
 }