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

Failure to parse xml attributes in tags #9

Closed
roticv opened this issue Aug 28, 2016 · 4 comments
Closed

Failure to parse xml attributes in tags #9

roticv opened this issue Aug 28, 2016 · 4 comments

Comments

@roticv
Copy link

roticv commented Aug 28, 2016

It looks like ooxi/xml.c is unable to parse attributes in tags. It fails with "xml_parser_error at 50:25: xml_parse_node::tag missmatch" and it is because it assumes that the value of tag_open = tag_close without stripping out the attributes before comparing.

@ooxi
Copy link
Owner

ooxi commented Sep 4, 2016

You are correct, parsing XML with attributes is not supported by xml.c

Implementing attribute parsing shouldn't be that complicated but is currently not in the scope of this XML subset parser (nevertheless, patches are welcome!). The API could be extended by xml_node_attribute very cleanly.

That being said, I'm leaving this ticket open as an reminder to specify the XML subset supported by xml.c :)

@devendranaga
Copy link

it cannot parse comments as well.. if i have comments in the document, it crashes with a coredump..

@ooxi
Copy link
Owner

ooxi commented Sep 26, 2018

@devnaga thank you very much for your interest in this project! Currently parsing comments is not supported, but you are correct it should be (they should be stripped).

I will open a separate issue for that

@ooxi
Copy link
Owner

ooxi commented Oct 11, 2019

Thanks to the work of @Molorius and @Isty001, xml.c can now also parse arguments. Feel free to reopen this issue if you have any more questsions :-)

@ooxi ooxi closed this as completed Oct 11, 2019
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

3 participants