[Tile-serving] [openstreetmap/osm2pgsql] Use fmt library instead of boost::format (#1004)

Jochen Topf notifications at github.com
Fri Nov 29 07:14:39 UTC 2019


joto commented on this pull request.



>              }
             for (size_t i = 0; i < hstore_columns.size(); ++i) {
-                m_sql_conn->exec("CREATE INDEX ON %1% USING GIN (\"%3%\") %4%",
-                                 m_target->name, i, hstore_columns[i],
-                                 (table_space_index
-                                      ? "TABLESPACE " + table_space_index.get()
-                                      : ""));
+                m_sql_conn->exec(
+                    "CREATE INDEX ON {} USING GIN (\"{}\") {}"_format(
+                        m_target->name, i, hstore_columns[i],

With the `_format` this should have been detectable at compile time.

-- 
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/1004#discussion_r352003850
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20191128/754ec086/attachment.html>


More information about the Tile-serving mailing list