search

serde-deprecated / xml

No longer maintained
35 stars 9 forks source link

Add test for forwarded namespace. #10

Closed winding-lines closed 8 years ago

winding-lines commented 8 years ago

This is documenting the current behavior of forwarded namespaces, my code depends on it and I want to make sure this is the right approach.

oli-obk commented 8 years ago

Glad that this is the behaviour you expect, but I don't understand what you mean by forwarding namespaces. Could you elaborate? Note: I have never used namespaces in xml.

winding-lines commented 8 years ago

I took that expression from the lexer. There is a comment about forwarding the namespace.

I think that all the schemas could be used to generate the rust code, instead of the programmer writing them manually. I believe that this is how some of the Java implementations are working. I do not need that feature for this one project and it is unclear to me how popular it is to generate code from xml schema nowadays.

This PR just documents the way it works now, I frequently look at tests when the documentation is not clear on a specific point.

What is your take on xml schemas?

Marius

On Jan 31, 2016, at 1:09 AM, Oliver Schneider notifications@github.com wrote:

Glad that this is the behaviour you expect, but I don't understand what you mean by forwarding namespaces. Could you elaborate? Note: I have never used namespaces in xml.

— Reply to this email directly or view it on GitHub.

winding-lines commented 8 years ago

FYI, I am trying to write a graphml parser, on top of serde-xml - thanks for your work on this!

Right now I am still trying to get the tests to pass.

https://github.com/winding-lines/graphml

oli-obk commented 8 years ago

Haha ok. Good that I know what I wrote..I'm not a fan of schemas, they are underdefined and there are too many incompatible implementations.

Cool project. Let me know if you need anything from serde-xml