Add "stacker" feature to band-aid common stack overflows (Kampfkarren…

…#260)

* Stacker feature

* Move clippy to its own test

.github/workflows/clippy.yml

@@ -0,0 +1,19 @@
+name: Clippy
+on:
+  push:
+    branches:
+    - main
+  pull_request:
+    branches:
+    - main
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: Clippy (default)
+      run: |
+        cargo clippy -- -D warnings
+    - name: Clippy (all features)
+      run: |
+        cargo clippy --all-features -- -D warnings

.github/workflows/test.yml

@@ -10,7 +10,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v1
+    - uses: actions/checkout@v3
     - name: Test (default features)
       run: |
         cd full-moon
@@ -39,12 +39,6 @@ jobs:
       run: |
         cd full-moon
         cargo test --no-default-features --features serde
-    - name: Clippy (default)
-      run: |
-        cargo clippy -- -D warnings
-    - name: Clippy (all features)
-      run: |
-        cargo clippy --all-features -- -D warnings
     - name: Rustfmt
       run: |
         cargo fmt -- --check

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## Unreleased
 ### Added
 - `full_moon::Error` and `full_moon::ast::Ast` now implement Serialize and Deserialize.
+- Added optional "stacker" feature which uses the [stacker](https://docs.rs/stacker/latest/stacker/index.html) crate to conditionally expand stack size to avoid stack overflows, a known problem with full-moon.
 
 ### Fixed
 - Support instantiated generics with no parameters, e.g. `Foo<>`

full-moon/Cargo.toml

@@ -32,6 +32,7 @@ logos = "0.12.0"
 paste = "0.1"
 serde = { version = "1.0", features = ["derive", "rc"], optional = true }
 smol_str = { version = "0.1.17", features = ["serde"] }
+stacker = { version = "0.1.15", optional = true }
 
 [dev-dependencies]
 criterion = "0.2"

full-moon/src/ast/parser_util.rs

@@ -109,6 +109,11 @@ macro_rules! define_parser {
                 &self,
                 state: ParserState<'a>,
             ) -> Result<(ParserState<'a>, $node), InternalAstError> {
+                #[cfg(feature = "stacker")]
+                if true {
+                    return stacker::maybe_grow(32 * 1024, 1024 * 1024, || $body(self, state));
+                }
+
                 $body(self, state)
             }
         }