search

georust / gpx

Rust read/write support for GPS Exchange Format (GPX)
https://crates.io/crates/gpx
MIT License
98 stars 44 forks source link

Determine XML library to base this on #1

Closed frewsxcv closed 7 years ago

frewsxcv commented 9 years ago

No need to reinvent the wheel with our own XML parser. I'm sure there are already a few, which one should we use?

groteworld commented 9 years ago

Option 1: netvl/xml-rs seems like a decent library with a better start at documentation than others I've seen, and has a group of devs. Was 1.0 alpha ready hours after release. Has limitations but is near xml-1.0 compliment. Check out the issues for some of these limitations

frewsxcv commented 9 years ago

Yeah, at least for now, that looks like a pretty good choice. Depending on when this particular project (rust-gpx) starts, new libraries might be available providing us with a greater selection

frewsxcv commented 9 years ago

There's also https://github.com/Florob/RustyXML

shivanth commented 7 years ago

Looks like RustyXML is not active any more. How about https://github.com/netvl/xml-rs ?

frewsxcv commented 7 years ago

https://github.com/tafia/quick-xml is also an option

shivanth commented 7 years ago

Looks like there are a lot of xml-libraries around. Is there one in std library or something ? Spoilt for choices :-)

shivanth commented 7 years ago

Is there any high level architecture for the georust project as such, so as to allow interoperability between the libraries. I am looking at implementing a reader and writer for GPX files .

frewsxcv commented 7 years ago

Currently, the idea is for https://github.com/georust/rust-geo to be the common geographic primitives connecting all the GeoRust projects. If you have suggestions for improvement, let me know :)

brendanashworth commented 7 years ago

I've just pushed the first commit that implements a basic parser: https://github.com/georust/rust-gpx/commit/67ca09b98e980cb042762b2db431aaa2bd2a9d51. Right now I've chosen to use xml-rs, but AFAIK it's not the fastest available parser. We can consider changing in the future, but for now I think this can be closed.