Move commiting of clones into process_queue()

This not only consolidates common code, it also has the effect of moving
the commit inside the code covered by the timer, which should result in
more realistic timer numbers.

Author: joto

src/osmdata.cpp

@@ -272,6 +272,12 @@ struct pending_threaded_processor : public pending_processor
                 throw;
             }
         }
+
+        for (auto const &clone : m_clones) {
+            for (auto const &clone_output : clone) {
+                clone_output->commit();
+            }
+        }
     }
 
     void process_ways() override
@@ -293,12 +299,6 @@ struct pending_threaded_processor : public pending_processor
                 stderr, "{} Pending ways took {}s at a rate of {:.2f}/s\n",
                 ids_queued, timer.elapsed(), timer.per_second(ids_queued));
         }
-
-        for (auto const &clone : m_clones) {
-            for (auto const &clone_output : clone) {
-                clone_output.get()->commit();
-            }
-        }
     }
 
     void process_relations() override
@@ -329,8 +329,6 @@ struct pending_threaded_processor : public pending_processor
                  original_output != m_outputs.end() &&
                  clone_output != clone.end();
                  ++original_output, ++clone_output) {
-                //done copying rels for now
-                clone_output->get()->commit();
                 //merge the expire tree from this threads copy of output back
                 original_output->get()->merge_expire_trees(clone_output->get());
             }