-
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
implement defparser macro #4
Comments
My thoughts: I just haven't thought deeply about it yet :) |
Lol. I'll investigate more deeply. Knock some code together as time permits. I think it's possible. You'd just have to port or re-implement most of the instaparse.core ns (which is not that big). |
Removing the limitation that parsers must be called with parse will be hard without support for implementing IFn on records or types. A simple |
Closing.
|
Hi, I was thinking about the defparser macro. Of course it couldn't possibly have worked inside the pod as it's defining a var in a different process :)
However, we should be able to implement it in instaparse-bb, perhaps starting with a simple version then moving to one which splits the work appropriately between compile time and runtime as instaparse itself does.
Possibly we could go further and have Parser as a record that implements Ifn (like instaparse) and so remove the limitation that parsers must be called with
parse
.@borkdude you must have considered this when you implemented the instaparse pod and instaparse-bb, so it would be good to hear your thoughts.
The text was updated successfully, but these errors were encountered: