-
Notifications
You must be signed in to change notification settings - Fork 448
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
Lexical error when parsing Integer values #360
Comments
…` / `examples/issue-360`
Hm, odd. This does work with my own jison 0.6.0-191 (npm: jison-gho). You might want to go and have a look at the internals, see what jison generated as code for your lexer, in particular the table with the regexes for your lexer rules. (The lexer is always the bottom half of the generated js parser/lexer source code file; the lexer tables are near the bottom of the file.) |
I ended up switching to nearleyjs which works fine for me. So a contributor can decide if he wants to close this issue, or try to fix it. Just curious, why did you chose to fork this library instead of contributing to it? |
I did contribute as well, but had specific needs for my own production
environment when swift turn-around / speed of development was a
requirement.
Managing a slew of single-subject/independent pullreqs (extracting them in
such a way as to make them independent, setting up and then tracking each
pullreq) next to the CTO day job wasnt a realistic choice re time costs, so
it started with a few pullreqs, got complex pretty darn fast and now you
have #338.
I'ld love to hear from @zaach if/when/how he wants to merge (some of?) the
work done some day, but its not just trivial changes under the hood and
clearly his needs and time constraints and mine re jison were (are?) quite
different at the time when i worked/work on jison a lot due to our/my needs
for it at the day job.
So i settled for the next best option: a fork. I dont like it, but its the
best option available under the circumstances.
I do a lot of work in bursts, which does not mix well with the
meditate-on-a-mountain management model that (by necessity) applies to a
lot of OSS as its volunteer driven rather than business driven. I like OSS
a lot, prefer it above all else, but the way of cooperating where people
with very diverse needs and availability work on a project is still a tough
nut I haven't cracked to my satisfaction. (note that i say "work" rather
than "contribute": this is about how to process change at a rate and depth
beyond the trivial)
Given my own limitations, I've developed a personal process where I can
track & merge others' work pretty quickly (sans the need for pullreq) when
my time allows spending attention on the given project, so I settled for
that, despite the drawbacks, until a merge is viable.
If you like what has been done, you might consider having a look at how to
mix this into the mainline. That's not my talent and it's much appreciated
if you can apply yours to it.
#338 is there for that reason: I dont want a fork, mine just happens to be
one.
…On 13 Sep 2017 9:45 am, "Jan Peter" ***@***.***> wrote:
I ended up switching to nearleyjs which works fine for me. So a
contributor can decide if he wants to close this issue, or try to fix it.
Just curious, why did you chose to fork this library instead of
contributing to it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#360 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAYkHnM1wRVJPnH5GHR-7PNwIUTvXB_Yks5sh4gFgaJpZM4PPcP3>
.
|
Thanks for the https://github.com/Hardmath123/nearley reference, by the way. |
Well, thanks for that detailed explanation! I would've also taken that route 👍 |
Wow, thanks for this discussion guys. I have been wondering about your fork for a while @GerHobbelt. Just installed it and trying it out in a few existing projects. |
I'm currently struggling with a strange problem. I want to parse the following snipped:
When I create my grammar and try to parse this snipped, I get the following error message:
The funny thing is, if I paste my grammar into a tool like this and choose a LRLA(1) grammar as parser type i get no errors. That's the way I create my grammar on my local machine:
That's my grammar as json:
I'm using the latest jison version
0.4.18
The text was updated successfully, but these errors were encountered: