search

dalance / nom-recursive

Extension of nom to handle left recursion
Apache License 2.0
16 stars 4 forks source link

Fix recursive parsers with inputs not named "s" #7

Closed fbenkstein closed 7 months ago

fbenkstein commented 2 years ago

Not naming the input of a #[recursive_parser] function s would cause an error like this:

  error[E0425]: cannot find value `s` in this scope
    --> nom-recursive/tests/test.rs:13:1
     |
  13 | #[recursive_parser]
     | ^^^^^^^^^^^^^^^^^^^ not found in this scope
     |

This fix properly uses #ident instead of hardcoded s.

spoutn1k commented 7 months ago

This is so essential why is it not merged

dalance commented 7 months ago

Sorry. I missed this PR. I'll merge this and release new version.

spoutn1k commented 7 months ago

No issue thank you for acting so fast !