Please expose a function to get the current position when parsing about xml-rs HOT 6 CLOSED

It is supposed that error messages should provide proper location information, and there is some infrastructure for that in the parser, but I think it is somewhat broken. I want to rework it one day. Exposing it is probably a good idea, but the proper way to do it also should be thought out properly because of event-based parsing approach.

I'll add it to my to-do list.

from xml-rs.

Yes, error messages seem to work as expected. I was more thinking about making this information available to the code that invokes the parser to show in errors caused by program logic not parsing logic, i.e., the parsed xml is syntactically correct but is not what is expected by the program. Findind out which of the thousands of tags in an xml document is formatted incorrectly without having to go through all of them by hand is very useful.

Thanks for looking into this!

from xml-rs.

Yes, I understand what you mean. As I said, this positioning code is not exactly correct, so I want to fix it first and then expose it to external consumers.

from xml-rs.

@netvl
So what's the plan on this? Should an event be carrying its position or should the parser provide the position of the last event?

from xml-rs.

@gkoz, I don't have any plans right now. I'm very sorry, but I'm very busy at my main job and I'm also working on my PhD thesis and several articles, all at the same time, so I just physically cannot work on side projects now :(

As a general thought, I don't want to add additional fields to events. Either the parser should provide a method to access the current position or there should be another iterator which returns (Event, Position) where Position: common::HasPosition.

from xml-rs.

@gkoz Thanks a lot!

from xml-rs.