[Tile-serving] [openstreetmap/osm2pgsql] Various cleanups/fixes found by clang-tidy (PR #1961)

Jochen Topf notifications at github.com
Mon May 15 06:58:10 UTC 2023


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

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

-- Commit Summary --

  * Avoid implicit widening conversions
  * Fix: Use uint32 for zoom value
  * Make conversion from integers to double explicit
  * Call static function logger::init_thread without instance
  * Always use explicit for constructors
  * Split up large test cases for code readability
  * Increasing threshold for code complexity warning in clang-tidy

-- File Changes --

    M .clang-tidy (4)
    M src/gen/gen-tile.hpp (2)
    M src/gen/osm2pgsql-gen.cpp (2)
    M src/middle-ram.hpp (2)
    M src/node-locations.hpp (2)
    M src/ordered-index.hpp (4)
    M src/pgsql.hpp (2)
    M src/progress-display.cpp (2)
    M src/thread-pool.cpp (2)
    M src/util.cpp (4)
    M src/wkb.cpp (4)
    M tests/test-db-copy-mgr.cpp (254)
    M tests/test-lua-utils.cpp (8)
    M tests/test-middle.cpp (3)
    M tests/test-node-locations.cpp (3)

-- Patch Links --

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

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

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


More information about the Tile-serving mailing list