[Tile-serving] [osm2pgsql] Use range-based loops in geometry-builder (#406)

[Paul Norman]

> -                            else if (polys[k].polygon->intersects(polys[j].polygon) || polys[k].polygon->touches(polys[j].polygon))
> -                            {
> -                                // FIXME: This code does not work as intended
> -                                // It should be setting the polys[k].ring in order to update this object
> -                                // but the value of polys[k].polygon calculated is normally NULL
> -
> -                                // Add polygon this polygon (j) to k since they intersect
> -                                // Mark ourselfs to be dropped (2), delete the original k
> -                                Geometry* polyunion = polys[k].polygon->Union(polys[j].polygon);
> -                                delete(polys[k].polygon);
> -                                polys[k].polygon = dynamic_cast<Polygon*>(polyunion);
> -                                polys[j].iscontained = 2; // Drop
> -                                istoplevelafterall = 2;
> -                                break;
> -                            }
> -#endif

This code has been commented out since it was written, 6-7 years ago.

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/406/files#r34955962
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20150719/e0aa7442/attachment.html>