[Tile-serving] [openstreetmap/osm2pgsql] Some cleanup for output-pgsql members (#907)

Sarah Hoffmann notifications at github.com
Sun Feb 10 08:57:14 UTC 2019


This PR cleans up two things:

* Convert tables member to std::array. The size of the array is known at compile time and does not change, so a vector is not necessary here.
* export_list is moved to the transform classes. The style file is only used by output-pgsql to create the columns. During the actual import it is only handed to the transforms on every filter call. Moving the export_list to transforms makes the filter calls more more readable and also gives us the freedom to change the style description format independently for the transforms.

Have a look at the individual commit for each change.

You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * output-pgsql: use fixed array for tables
  * move export_list into c transform

-- File Changes --

    M output-multi.cpp (29)
    M output-multi.hpp (3)
    M output-pgsql.cpp (55)
    M output-pgsql.hpp (6)
    M tagtransform-c.cpp (22)
    M tagtransform-c.hpp (11)
    M tagtransform-lua.cpp (26)
    M tagtransform-lua.hpp (12)
    M tagtransform.cpp (6)
    M tagtransform.hpp (11)
    M tests/test-options-parse.cpp (4)

-- Patch Links --

https://github.com/openstreetmap/osm2pgsql/pull/907.patch
https://github.com/openstreetmap/osm2pgsql/pull/907.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/907
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20190210/202f14b3/attachment-0001.html>


More information about the Tile-serving mailing list