[Tile-serving] [openstreetmap/osm2pgsql] multi-output: reintroduce line spliting (#755)

[Sarah Hoffmann]

lonvia commented on this pull request.



>  {
     wkbs_t ret;
 
-    double const split_at = m_proj->target_latlon() ? 1 : 100 * 1000;
+    bool do_split = split_at > 0.0;

I would expect that comparison against zero is reasonably safe. I was considering using NaN but it did feel like a misusing the value. The other option is to add do_split as an additional parameter, which would be fine to me as well.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/755#discussion_r115206033
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20170508/b8ef44de/attachment.html>