Parser's line/column not reported on parser's errors about pybison

[arquer]

Hello,

if possible, could you just point me in the direction to solve this issue? If I knew where to start I could fix it myself.

[smvv]

@arquer there are multiple path to the syntax error messages. One of them is https://github.com/smvv/pybison/blob/master/src/python/__init__.py#L348

The other one is generated and put into the Bison grammar file: https://github.com/smvv/pybison/blob/master/src/pyrex/bison_.pyx#L416-L442

[arquer]

Yes, I saw those functions @smvv, however as I pointed out pybison keeps reporting all syntax errors with all zeros.. I even tried to define YYLLOC_DEFAULT (I don't know if that is actually needed or if bison already defines it by default).
I was thinking about modifying the python callbacks to include a new argument "locations" (besides target, names, option and values. The locations argument could be a list of pointers pointing to Bison's @n and @$ structures (yylloc structs if I remember correctly), that way the user could assign line numbers from one production to another. The problem I see is how to automatically assign location information to terminals.
What do you think?

[arquer]

I would really like your input on this... Just to know if I am going about it allright.

So far I am finding the package very usefull but I need to overcome some limitations in order to be able to use it. I will eventually make the parser I am doing (SystemVerilog, not Verilog, which is much easier) open-source.
So far my main concerns are:

• Get location of parser when an error occurs
• Get it to work under windows. At least the actual parsing, bison/flex files could be generated on a linux machine
• Allow multiple instance of the Parser to be ran concurrently.

[arquer]

I got the line number reporting working in the parser. I am sorry but I did not know all I had to do is define a YY_USER_ACTION.
Now as I stated above I am trying to get that line information available in my python callbacks so I can include that in my AST.

[arquer]

I have solved this issue, I have successfully passed all location information computed by bison to my python callbacks adding another argument.