[Tile-serving] [osm2pgsql-dev/osm2pgsql] Various small cleanups (PR #2202)

Jochen Topf notifications at github.com
Sat Jun 15 18:05:19 UTC 2024


Found by clang-tidy. See individual commits.
You can view, comment on, or merge this pull request online at:

  https://github.com/osm2pgsql-dev/osm2pgsql/pull/2202

-- Commit Summary --

  * Remove superfluous c_str(), decode_hex() parameter is a string_view
  * Avoid class member that is a reference
  * Replace short variable/member name "ls" by "linestring"
  * Rename function parameter to avoid confusing parameters
  * Use emplace_back() instead of push_back()
  * Remove unused code

-- File Changes --

    M src/flex-lua-index.cpp (2)
    M src/gen/osm2pgsql-gen.cpp (8)
    M src/geom-functions.cpp (38)
    M src/output-pgsql.cpp (4)
    M tests/test-wkb.cpp (4)

-- Patch Links --

https://github.com/osm2pgsql-dev/osm2pgsql/pull/2202.patch
https://github.com/osm2pgsql-dev/osm2pgsql/pull/2202.diff

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

Message ID: <osm2pgsql-dev/osm2pgsql/pull/2202 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20240615/933de095/attachment.htm>


More information about the Tile-serving mailing list