From c5a35e388ca2ab34937b654c09129b05d91982dc Mon Sep 17 00:00:00 2001 From: Hugo Duncan Date: Fri, 20 Feb 2015 13:59:14 -0500 Subject: [PATCH] Refactor default_value to use if-let --- serde2/serde2_macros/src/lib.rs | 36 ++++++++++++++++----------------- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/serde2/serde2_macros/src/lib.rs b/serde2/serde2_macros/src/lib.rs index 90145520..e0c8caf7 100644 --- a/serde2/serde2_macros/src/lib.rs +++ b/serde2/serde2_macros/src/lib.rs @@ -6,6 +6,7 @@ extern crate rustc; use syntax::ast::{ Ident, MetaItem, + MetaItem_, Item, Expr, MutMutable, @@ -854,24 +855,23 @@ fn declare_map_field_deserializer( fn default_value(field: &ast::StructField) -> bool { field.node.attrs.iter() .any(|sa| - match &sa.node.value.node { - &ast::MetaItem_::MetaList(ref n, ref vals) => { - if n == &"serde" { - vals.iter() - .map(|mi| - match &mi.node { - &ast::MetaItem_::MetaWord(ref n) => { - n == &"default" - }, - _ => false - }) - .any(|x| x) - } else { - false - } - }, - _ => false - } ) + if let MetaItem_::MetaList(ref n, ref vals) = sa.node.value.node { + if n == &"serde" { + vals.iter() + .map(|mi| + if let MetaItem_::MetaWord(ref n) = mi.node { + n == &"default" + } else { + false + }) + .any(|x| x) + } else { + false + } + } + else { + false + }) } fn declare_visit_map(