[Tile-serving] [openstreetmap/osm2pgsql] Misc code cleanup (PR #1652)

Jochen Topf notifications at github.com
Mon Mar 14 11:17:28 UTC 2022


Also see individual commits.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Fix various small issues found by clang-tidy and disable some warnings
  * Reorder struct members to use less padding
  * Modernize code: Use pass-by-value and std::move()
  * Avoid copying options inside output_t and access them through getter

-- File Changes --

    M .clang-tidy (10)
    M src/flex-table-column.cpp (6)
    M src/flex-table-column.hpp (2)
    M src/format.hpp (3)
    M src/gazetteer-style.hpp (4)
    M src/middle-pgsql.cpp (20)
    M src/middle-pgsql.hpp (5)
    M src/options.cpp (2)
    M src/options.hpp (80)
    M src/output-flex.cpp (22)
    M src/output-gazetteer.cpp (14)
    M src/output-gazetteer.hpp (4)
    M src/output-null.cpp (2)
    M src/output-pgsql.cpp (61)
    M src/output.cpp (7)
    M src/output.hpp (8)
    M src/pgsql.cpp (8)
    M src/table.cpp (12)
    M src/table.hpp (7)
    M src/taginfo.hpp (4)
    M src/tagtransform-c.cpp (44)
    M src/tagtransform-c.hpp (2)
    M src/wkb.cpp (14)
    M tests/test-expire-tiles.cpp (14)
    M tests/test-middle.cpp (6)
    M tests/test-output-gazetteer.cpp (2)
    M tests/test-output-pgsql-style-file.cpp (4)
    M tests/test-parse-osmium.cpp (2)

-- Patch Links --

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

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

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


More information about the Tile-serving mailing list