From fbfe70e6ab26c0cccb5fbbf5b805eecb751cb576 Mon Sep 17 00:00:00 2001 From: dileepb Date: Tue, 23 Feb 2016 21:18:07 +0530 Subject: #31820 - Utilize `if..let` instead of single `match` branch --- src/libstd/net/addr.rs | 5 ++--- src/libstd/net/parser.rs | 5 ++--- src/libstd/sys/windows/thread_local.rs | 5 ++--- 3 files changed, 6 insertions(+), 9 deletions(-) (limited to 'src/libstd') diff --git a/src/libstd/net/addr.rs b/src/libstd/net/addr.rs index 89c51c70843..87d5dc20e9f 100644 --- a/src/libstd/net/addr.rs +++ b/src/libstd/net/addr.rs @@ -467,9 +467,8 @@ impl ToSocketAddrs for str { type Iter = vec::IntoIter; fn to_socket_addrs(&self) -> io::Result> { // try to parse as a regular SocketAddr first - match self.parse().ok() { - Some(addr) => return Ok(vec![addr].into_iter()), - None => {} + if let Some(addr) = self.parse().ok() { + return Ok(vec![addr].into_iter()); } macro_rules! try_opt { diff --git a/src/libstd/net/parser.rs b/src/libstd/net/parser.rs index 46a0309dbb5..63eee6bcfde 100644 --- a/src/libstd/net/parser.rs +++ b/src/libstd/net/parser.rs @@ -66,9 +66,8 @@ impl<'a> Parser<'a> { fn read_or(&mut self, parsers: &mut [Box Option + 'static>]) -> Option { for pf in parsers { - match self.read_atomically(|p: &mut Parser| pf(p)) { - Some(r) => return Some(r), - None => {} + if let Some(r) = self.read_atomically(|p: &mut Parser| pf(p)) { + return Some(r); } } None diff --git a/src/libstd/sys/windows/thread_local.rs b/src/libstd/sys/windows/thread_local.rs index db2ad1d89c4..386d21a04c7 100644 --- a/src/libstd/sys/windows/thread_local.rs +++ b/src/libstd/sys/windows/thread_local.rs @@ -69,9 +69,8 @@ static mut DTORS: *mut Vec<(Key, Dtor)> = ptr::null_mut(); pub unsafe fn create(dtor: Option) -> Key { let key = c::TlsAlloc(); assert!(key != c::TLS_OUT_OF_INDEXES); - match dtor { - Some(f) => register_dtor(key, f), - None => {} + if let Some(f) = dtor { + register_dtor(key, f); } return key; } -- cgit 1.4.1-3-g733a5