[Tile-serving] [openstreetmap/osm2pgsql] Add implementation of simplify function for multilinestrings (PR #1782)

Jochen Topf notifications at github.com
Thu Sep 29 12:11:47 UTC 2022


We already have an implementation for linestring, which is slightly improved in this commit. Simplification for anything other than (multi)linestring is not supported.

In the first commit there is also a refactor of the `fill_point_list()` function which makes the code more readable and consistent.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Refactor: Make fill_point_list return a bool showing validity
  * Add remove_duplicates() and is_valid() functions for linestrings
  * Implement simplify for multilinestrings

-- File Changes --

    M src/geom-from-osm.cpp (25)
    M src/geom-functions.cpp (47)
    M src/geom.cpp (6)
    M src/geom.hpp (16)
    M tests/test-geom-linestrings.cpp (38)
    M tests/test-geom-multilinestrings.cpp (42)

-- Patch Links --

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

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/1782
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/osm2pgsql/pull/1782 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20220929/5efc72cf/attachment.htm>


More information about the Tile-serving mailing list