Allow stacking ! operators.

Cargo.toml

@@ -17,9 +17,10 @@ license = "MIT OR Apache-2.0"
 include = [
     "**/*.rs",
     "scripts/*.rhai",
+    "**/*.md",
     "Cargo.toml"
 ]
-keywords = [ "scripting" ]
+keywords = [ "scripting", "scripting-engine", "scripting language", "embedded" ]
 categories = [ "no-std", "embedded", "wasm", "parser-implementations" ]
 
 [dependencies]

RELEASES.md

@@ -19,6 +19,7 @@ Bug fixes
 
 * Parameters passed to plugin module functions were sometimes erroneously consumed. This is now fixed.
 * Fixes compilation errors in `metadata` feature build.
+* Stacking `!` operators now work properly.
 
 New features
 ------------

src/parser.rs

@@ -1338,7 +1338,7 @@ fn parse_unary(
         Token::Bang => {
             let pos = eat_token(input, Token::Bang);
             let mut args = StaticVec::new();
-            let expr = parse_primary(input, state, lib, settings.level_up())?;
+            let expr = parse_unary(input, state, lib, settings.level_up())?;
             args.push(expr);
 
             let op = "!";

tests/internal_fn.rs

@@ -62,9 +62,9 @@ fn test_internal_fn_overloading() -> Result<(), Box<EvalAltResult>> {
         *engine
             .compile(
                 r"
-                fn abc(x) { x + 42 }
+                    fn abc(x) { x + 42 }
-                fn abc(x) { x - 42 }
+                    fn abc(x) { x - 42 }
-            "
+                "
             )
             .expect_err("should error")
             .0,

tests/not.rs

@@ -12,8 +12,7 @@ fn test_not() -> Result<(), Box<EvalAltResult>> {
     #[cfg(not(feature = "no_function"))]
     assert_eq!(engine.eval::<bool>("fn not(x) { !x } not(false)")?, true);
 
-    // TODO - do we allow stacking unary operators directly? e.g '!!!!!!!true'
+    assert_eq!(engine.eval::<bool>("!!!!true")?, true);
-    assert_eq!(engine.eval::<bool>("!(!(!(!(true))))")?, true);
 
     Ok(())
 }