extend geo url scheme
@pointhi
Submitted by Thomas Pointhuber Link to original bug (#770633)
Description
Created attachment 334514 initial patch set to extend geo url scheme
similar patch can be found here: 737979
Adding support for unknown parameters inside RFC5870 geo url
When looking onto the RFC5870, it's possible to add custom parameters. Currently geocode-glib returns an error when this happens
Extend Android geo url support
OSM for example returns Android defined geo urls in the format "geo:latitude,longitude?z=zoom". My patch allows parsing of those urls too, even there is no zoom property implemented yet. I also added support for multiple arguments (like "?q=...&z=...")
Outstanding questions:
-
currently I have written the parameter splitter in a way it supports at maximum 256 arguments. (based on the old code). Probably using strchr which then alwayssearchs for the next argument would be a saner implementation
-
would there be concerns about adding support for zoom levels?
-
what about url's like "geo:0,0?q=1600+Amphitheatre+Parkway%2C+CA" as stated in 737979? I didn't implemented yet because of some concerns about returning only a description (LAT & LON would be undefined)
patch can also be found on github:
https://github.com/pointhi/geocode-glib/tree/set_from_uri_improvements
Patch 334514, "initial patch set to extend geo url scheme":
extend_geo_url_scheme.patch