[Tile-serving] [osm2pgsql-dev/osm2pgsql] Refactor: Simplifiy code by using std::variant for copy mgr (PR #2209)

Sarah Hoffmann notifications at github.com
Tue Jul 9 09:01:55 UTC 2024


@lonvia commented on this pull request.



>      }
 
     void flush()
     {
         // flush current buffer if there is one
         if (m_current) {
-            m_processor->add_buffer(std::move(m_current));
+            m_processor->send_command(
+                db_cmd_copy_delete_t<DELETER>{std::move(m_current)});

Does m_current really need wrapping here?

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

Message ID: <osm2pgsql-dev/osm2pgsql/pull/2209/review/2165515148 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20240709/6133fd74/attachment.htm>


More information about the Tile-serving mailing list