[Tile-serving] [osm2pgsql-dev/osm2pgsql] Consistent naming and type for "Lua class names" (PR #2340)

Jochen Topf notifications at github.com
Tue Jun 3 08:18:55 UTC 2025


* Using uppercase, because they are constants
* Using the word "CLASS" because it is easier to understand then the generic "name" or the Luaish "metatable"
* Using char* not std::string_view, because its simpler use (no .data() needed)

This is for all the Lua "classes" that we use in flex config files:

* ExpireOutput
* Geometry
* Locator
* OSMObject
* Table
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Consistent naming and type for "Lua class names"

-- File Changes --

    M src/flex-lua-expire-output.cpp (4)
    M src/flex-lua-expire-output.hpp (2)
    M src/flex-lua-geom.cpp (8)
    M src/flex-lua-locator.cpp (4)
    M src/flex-lua-locator.hpp (2)
    M src/flex-lua-table.cpp (6)
    M src/flex-lua-table.hpp (2)
    M src/gen/osm2pgsql-gen.cpp (2)
    M src/output-flex.cpp (18)

-- Patch Links --

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

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

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


More information about the Tile-serving mailing list