[Tile-serving] [osm2pgsql] Remove the global text-tree string storage and make keyval a real class (#240)

Sarah Hoffmann notifications at github.com
Sun Dec 28 21:18:53 UTC 2014


The list implementation is still carried over from the C code. I'd love to see the keyvals use a simple std::vector eventually but at the moment much of the code depends on it being a linked list (doing a lot of chaining and unchaining of items, much more than necessary). So changing the container only makes sense when those parts are rewritten as well. I felt that this was too much for one pull request. It also might be better to get rid of the remaining hand-allocated char* strings before rewriting keyvals to have more readable code to start with.

---
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/240#issuecomment-68219832
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20141228/2d0a26b7/attachment.html>


More information about the Tile-serving mailing list