[Tile-serving] [openstreetmap/osm2pgsql] split tagtransform into two classes and an interface (#712)

Paul Norman notifications at github.com
Sat Mar 11 05:27:32 UTC 2017


pnorman commented on this pull request.

Overall looks good except for what the warning is about, but I'm wondering, do we want to change the C++ interface for tagtransform?

>  
-class tagtransform {
+class c_tagtransform_t : public tagtransform_t

Should these be in `tagtransform-c.hpp`/`tagtransform-lua.hpp` to match the cpp files?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/712#pullrequestreview-26416235
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20170310/84ed92ca/attachment.html>


More information about the Tile-serving mailing list