commit 59d4505749073f45db9a32d606b760eb9e07df69 Author: Corey Farwell coreyf@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); }