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

python ParseError reported location should calculate special character such as '\t' #20

Closed
jamesruan opened this issue Aug 22, 2017 · 7 comments

Comments

@jamesruan
Copy link

jamesruan commented Aug 22, 2017

Line 3: expected [A-Za-z0-9], "\t"
                ifindex 3
           ^

because there's 2 '\t' before ifindex, the actual location of error is somewhere later.

@jcoglan
Copy link
Owner

jcoglan commented Sep 2, 2017

Could you give an example grammar and input that produces this error?

@jamesruan
Copy link
Author

Try to parse error on lines with "\t". You may construct such a case faster than me.

@jcoglan
Copy link
Owner

jcoglan commented Sep 2, 2017

Sorry, I was asking for an example so I can make sure I've understood the problem you're seeing, because I was unsure from reading your initial report. For example, I don't know where the parser error is actually happening.

@jamesruan
Copy link
Author

jamesruan commented Sep 4, 2017

oh, the error in my example should be pointed like this

Line 3: expected [A-Za-z0-9], "\t"
                ifindex 3
                       ^

expecting a "\t", but met a " ".

@jcoglan
Copy link
Owner

jcoglan commented Sep 4, 2017

Thanks for explaining that. But I was mentioning that as one example of how, without an explicit example from you, I'm going to be guessing at the cause and solution to this problem.

The quickest way to make sure I can reproduce your problem would be if you put a grammar and input file on gist.github.com that cause this error, then I can begin to debug it.

@jamesruan
Copy link
Author

Sorry for that I would not provide such an example.

Because this library falls short of my expectations, I've already abandoned using it. Provide an example need extra time that I'm short of.

This issue raised simply try to make this a better one, even I'm not using it. Maybe somebody using it also found this issue will provide one.

@jcoglan
Copy link
Owner

jcoglan commented Sep 26, 2021

I'm closing this since @jamesruan has moved on from using this package.

@jcoglan jcoglan closed this as completed Sep 26, 2021
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

No branches or pull requests

2 participants