[tor-commits] [tor/master] Utilize `if..else` for switching on boolean values.

nickm at torproject.org nickm at torproject.org
Fri Jul 13 22:30:26 UTC 2018


commit 59d4505749073f45db9a32d606b760eb9e07df69
Author: Corey Farwell <coreyf at rwell.org>
Date:   Sun Jun 24 10:27:46 2018 -0400

    Utilize `if..else` for switching on boolean values.
---
 src/rust/protover/ffi.rs | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/rust/protover/ffi.rs b/src/rust/protover/ffi.rs
index f668dba80..ba156cb1b 100644
--- a/src/rust/protover/ffi.rs
+++ b/src/rust/protover/ffi.rs
@@ -105,9 +105,10 @@ pub extern "C" fn protocol_list_supports_protocol(
         Ok(n) => n.into(),
         Err(_) => return 0,
     };
-    match proto_entry.supports_protocol(&protocol, &version) {
-        false => return 0,
-        true  => return 1,
+    if proto_entry.supports_protocol(&protocol, &version) {
+        1
+    } else {
+        0
     }
 }
 
@@ -206,13 +207,16 @@ pub extern "C" fn protover_compute_vote(
     let mut proto_entries: Vec<UnvalidatedProtoEntry> = Vec::new();
 
     for datum in data {
-        let entry: UnvalidatedProtoEntry = match allow_long_proto_names {
-            true => match UnvalidatedProtoEntry::from_str_any_len(datum.as_str()) {
+        let entry: UnvalidatedProtoEntry = if allow_long_proto_names {
+            match UnvalidatedProtoEntry::from_str_any_len(datum.as_str()) {
                 Ok(n)  => n,
-                Err(_) => continue},
-            false => match datum.parse() {
+                Err(_) => continue
+            }
+        } else {
+            match datum.parse() {
                 Ok(n)  => n,
-                Err(_) => continue},
+                Err(_) => continue
+            }
         };
         proto_entries.push(entry);
     }





More information about the tor-commits mailing list