Skip to content
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

Make Parser instance available to postprocessors #203

Open
tjvr opened this issue Mar 11, 2017 · 3 comments
Open

Make Parser instance available to postprocessors #203

tjvr opened this issue Mar 11, 2017 · 3 comments

Comments

@tjvr
Copy link
Collaborator

tjvr commented Mar 11, 2017

How about in postprocessors binding this to the Parser? (Or making the Parser the fourth argument?) Massive potential for abuse, of course; but it would stop us from adding a potentially unbounded number of arguments to State :-)

For #201.

@EricMCornelius
Copy link

So, how do you anticipate getting the Parser object reference passed to the State.finish prototype method? Manual passing of the parameter to the invocation as an argument? That would require a reference on the Column. Or should that be also threaded through the Column.process?

@tjvr
Copy link
Collaborator Author

tjvr commented Mar 11, 2017

should that be also threaded through the Column.process?

Probably, to avoid the circular reference. :-)

@simonhildebrandt
Copy link
Contributor

This would be lovely. :)

swwu added a commit to swwu/nearley that referenced this issue Jul 3, 2023
swwu added a commit to swwu/nearley that referenced this issue Jul 3, 2023
swwu added a commit to swwu/nearley that referenced this issue Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants