[Tile-serving] [openstreetmap/osm2pgsql] Gazetteer: reorganize deletion of objects (#1031)

[Sarah Hoffmann]

There was still a case were a delete function was called on an unprepared buffer leading to a segfault. This PR attempts to fix that once and for all.

Move prepare() call into delete functions to make sure that it is really called. Also reorganise how the delete functions are called to ensure that only one delete is executed per object.

Also adds a test for full delete of a node.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * add test for deleting node in gazetteer output
  * gazetteer: reorganise deletion of objects

-- File Changes --

    M src/output-gazetteer.cpp (142)
    M src/output-gazetteer.hpp (45)
    M tests/test-output-gazetteer.cpp (13)

-- Patch Links --

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

-- 
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/1031
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20191215/880f3eda/attachment.html>