This branch attempts to cleanup and make the elements easier to work with. Mainly, it
uses type Int for IDs
makes arrays non-optional to remove the need for unwrapping
introduces a protocol (OverpassResponseElementsProviding) that can be used to look up related features, removing the need to depend on the whole OverpassResponse
fixes a potential (well, hypothetical) crash when the XML parser encounters a way without a ref
Last but not least, it renames OverpassEntity to OverpassElement to be more in line with the official OpenStreetMap lingo, and adds some basic documentation to the elements that should help developers who are new to OSM (like me) understand what the elements' purpose is.
What do you think? As always, feedback is much appreciated.
holyturt
commented
6 years ago
This branch attempts to cleanup and make the elements easier to work with. Mainly, it
Intfor IDsOverpassResponseElementsProviding) that can be used to look up related features, removing the need to depend on the wholeOverpassResponserefLast but not least, it renames
OverpassEntitytoOverpassElementto be more in line with the official OpenStreetMap lingo, and adds some basic documentation to the elements that should help developers who are new to OSM (like me) understand what the elements' purpose is.What do you think? As always, feedback is much appreciated.