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

Sarah Hoffmann notifications at github.com
Sat Mar 11 08:54:24 UTC 2017


std::move fixed.

> but I'm wondering, do we want to change the C++ interface for tagtransform?

I'm not sure what you mean.

If you are worried that this PR changes the interface, then that is not the case. I've only changed the name of the class to match our naming conventions, otherwise the public interface is the same.

If you mean that we should change the C++ interface further than what this PR does, then the answer is: absolutely. I just would like to have proper C++ code before doing any semantic changes.


-- 
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#issuecomment-285853779
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20170311/afd756eb/attachment.html>


More information about the Tile-serving mailing list