Merge pull request #480 from ltratt/manual_clippy_fixes

Manual clippy fixes.

Author: ratmice

cfgrammar/src/lib/yacc/parser.rs

@@ -823,7 +823,7 @@ impl YaccParser {
                 ':' => {
                     let k = j + ':'.len_utf8();
                     if k == self.src.len() || !self.src[k..].starts_with(':') {
-                        return Ok((j, self.src[i..j].to_string()));
+                        return Ok((j, self.src[i..j].trim().to_string()));
                     }
                     j += 2 * ':'.len_utf8();
                 }

lrpar/cttests/src/parseparam_copy.test

@@ -9,7 +9,12 @@ grammar: |
         // Since then we relaxed the bounds to require `Clone`.
         // This tests backwards compatibility of actions that
         // rely on the older copy bounds.
-        'INT' { (move |_| {})(p); check_copy(p); p + $lexer.span_str($1.unwrap().span()).parse::<u64>().unwrap() }
+        'INT' {
+            #[allow(clippy::redundant_closure_call)]
+            (move |_| {})(p);
+            check_copy(p);
+            p + $lexer.span_str($1.unwrap().span()).parse::<u64>().unwrap()
+        }
     ;
     %%
     fn check_copy<T: Copy>(_: T){}