From 7be39d725d6d251366d1cbd2566651b011cdb5a3 Mon Sep 17 00:00:00 2001 From: Corey Farwell Date: Wed, 20 Jan 2016 23:30:45 -0500 Subject: Address issues found by rust-clippy --- src/decoder/rustc_serialize.rs | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) (limited to 'src/decoder/rustc_serialize.rs') diff --git a/src/decoder/rustc_serialize.rs b/src/decoder/rustc_serialize.rs index 9d2644e..2f4fb09 100644 --- a/src/decoder/rustc_serialize.rs +++ b/src/decoder/rustc_serialize.rs @@ -10,7 +10,7 @@ impl rustc_serialize::Decoder for Decoder { type Error = DecodeError; fn read_nil(&mut self) -> Result<(), DecodeError> { match self.toml { - Some(Value::String(ref s)) if s.len() == 0 => {} + Some(Value::String(ref s)) if s.is_empty() => {} Some(Value::String(..)) => return Err(self.err(NilTooLong)), ref found => return Err(self.mismatch("string", found)), } @@ -158,7 +158,7 @@ impl rustc_serialize::Decoder for Decoder { Some(Value::Table(..)) => { let ret = try!(f(self)); match self.toml { - Some(Value::Table(ref t)) if t.len() == 0 => {} + Some(Value::Table(ref t)) if t.is_empty() => {} _ => return Ok(ret) } self.toml.take(); @@ -243,7 +243,7 @@ impl rustc_serialize::Decoder for Decoder { match self.toml { Some(Value::Array(ref mut arr)) => { arr.retain(|slot| slot.as_integer() != Some(0)); - if arr.len() != 0 { return Ok(ret) } + if !arr.is_empty() { return Ok(ret) } } _ => return Ok(ret) } @@ -262,12 +262,10 @@ impl rustc_serialize::Decoder for Decoder { }; let mut d = self.sub_decoder(Some(toml), ""); let ret = try!(f(&mut d)); - match d.toml { - Some(toml) => match self.toml { - Some(Value::Array(ref mut arr)) => arr[idx] = toml, - _ => {} - }, - _ => {} + if let Some(toml) = d.toml { + if let Some(Value::Array(ref mut arr)) = self.toml { + arr[idx] = toml; + } } Ok(ret) } @@ -290,7 +288,7 @@ impl rustc_serialize::Decoder for Decoder { let ret = try!(f(self, amt)); let leftover = mem::replace(&mut self.leftover_map, prev_map); self.cur_map = prev_iter; - if leftover.len() > 0 { + if !leftover.is_empty() { self.toml = Some(Value::Table(leftover)); } Ok(ret) @@ -317,9 +315,9 @@ impl rustc_serialize::Decoder for Decoder { if let Some(toml) = d.toml.take() { self.leftover_map.insert(key, toml); } - return ret + ret } - None => return Err(self.err(ExpectedMapElement(idx))), + None => Err(self.err(ExpectedMapElement(idx))), } } -- cgit v1.2.3