[Tile-serving] [osm2pgsql] Ignore nodes with invalid coordinates. (#455)

[Matt Amos]

I hit this while processing a snapshot extracted from history which happened to contain [version 3 of node 7226378](http://www.openstreetmap.org/node/7226378/history), which has coordinates (83.4542914, 214.7483647). The API at the time didn't check this, and it has since been fixed, but it means that [the node parsing code](https://github.com/openstreetmap/osm2pgsql/blob/master/parse-osmium.cpp#L64-L65) will throw an exception.

(Notably, this doesn't kill the process, as it means the consumer thread waits on the Queue to finish processing, while the producer thread is waiting on the Queue to empty. But this is a separate issue.)

In this case it seems simple enough to just ignore any node which has an invalid location, which is effectively the same behaviour as setting the bbox to the world extent.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/osm2pgsql/pull/455

-- Commit Summary --

  * Ignore nodes with invalid coordinates.

-- File Changes --

    M parse-osmium.cpp (5)

-- Patch Links --

https://github.com/openstreetmap/osm2pgsql/pull/455.patch
https://github.com/openstreetmap/osm2pgsql/pull/455.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/455
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20151008/0e6f24c2/attachment.html>