Skip to content

Conversation

@zesterer
Copy link
Owner

No description provided.

@m-hugo
Copy link

m-hugo commented Mar 17, 2023

false positive? this https://github.com/m-hugo/hugoc/tree/f39632f5f9d1d1bb658d051764cf38eabd2b3018
panics on debug but runs fine on release (except for a surprising found end of input expected something else before the actual end of the file)
slapping a dozen memoized in other places doesn't remove that Recursive parser defined at src/main.rs:43:9 is left-recursive. Consider using .memoized() or restructuring this parser to be right-recursive.

@zesterer
Copy link
Owner Author

false positive?

Yep, this is why this isn't merged yet. The check assumes that parsers are functionally pure, but .memoized() messes with this assumption for implementation reasons. I've yet to come up with a reliable and efficient way around it. When I do, I'll try to get this merged.

@zesterer zesterer force-pushed the main branch 5 times, most recently from 38bb88c to 02a1373 Compare May 26, 2025 14:11
@zesterer zesterer force-pushed the main branch 2 times, most recently from 0f2b61a to e350fc6 Compare November 5, 2025 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants