Minor Clippy-suggested cleanups.

kaj · Apr 27, 2017 · 0ff9dcc · 0ff9dcc
1 parent eac50f1
commit 0ff9dcc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -229,8 +229,8 @@ pub enum SassItem {
     Each(String, Value, Vec<SassItem>),
     For {
         name: String,
-        from: Value,
-        to: Value,
+        from: Box<Value>,
+        to: Box<Value>,
         inclusive: bool,
         body: Vec<SassItem>,
     },
@@ -326,8 +326,8 @@ named!(for_loop<SassItem>,
                  body: body_block >>
                  (SassItem::For {
                      name: name,
-                     from: from,
-                     to: to,
+                     from: Box::new(from),
+                     to: Box::new(to),
                      inclusive: inclusive,
                      body: body
                  })));

diff --git a/src/valueexpression.rs b/src/valueexpression.rs
@@ -633,7 +633,7 @@ named!(simple_dqs_part<Value>,
                                Quotes::Double)));
 named!(nonempty_dqs_part<Value>,
        map!(verify!(escaped!(is_not!("\\\"#"), '\\', take!(1)),
-                    |s: &[u8]| s.len() > 0),
+                    |s: &[u8]| !s.is_empty()),
             |s| Value::Literal(unescape(from_utf8(s).unwrap()),
                                Quotes::Double)));
 
@@ -655,7 +655,7 @@ named!(simple_sqs_part<Value>,
                                Quotes::Single)));
 named!(nonempty_sqs_part<Value>,
        map!(verify!(escaped!(is_not!("\\'#"), '\\', take!(1)),
-                    |s: &[u8]| s.len() > 0),
+                    |s: &[u8]| !s.is_empty()),
             |s| Value::Literal(unescape(from_utf8(s).unwrap()),
                                Quotes::Single)));