[Tile-serving] [osm2pgsql-dev/osm2pgsql] Add suffix_t to all classes and enable naming checks in clang-tidy (PR #2389)

Jochen Topf notifications at github.com
Thu Aug 21 07:12:41 UTC 2025


Over the last months we have fixed some issues with inconsistent naming. This is the (for the time being) last such change adding `_t` to class name that don't have that already.

And this PR enables checks in clang-tidy to make sure we keep to our naming rules in the future.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Consistently use _t suffix for classes
  * Enable more checks in clang-tidy around naming

-- File Changes --

    M .clang-tidy (22)
    M src/geom-functions.cpp (36)
    M src/middle-pgsql.cpp (8)
    M src/osmdata.cpp (14)
    M src/wkb.cpp (8)
    M tests/test-flex-indexes.cpp (26)
    M tests/test-util.cpp (6)

-- Patch Links --

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

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

Message ID: <osm2pgsql-dev/osm2pgsql/pull/2389 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20250821/07add507/attachment-0001.htm>


More information about the Tile-serving mailing list