VM: vm::iterate_active_callstacks is only used once, so it's simpler if the code...

diff --git a/vm/code_block_visitor.hpp b/vm/code_block_visitor.hpp
index 6439629c97e9ac01e7cb6cd8bcfc7c8d4dfd7a3c..8cd7dab6d2b0b2d3155a2cf5a742ed392ecc4152 100644 (file)
--- a/vm/code_block_visitor.hpp
+++ b/vm/code_block_visitor.hpp
@@ -95,7 +95,11 @@ void code_block_visitor<Fixup>::visit_embedded_code_pointers(
 template <typename Fixup>
 void code_block_visitor<Fixup>::visit_context_code_blocks() {
   call_frame_code_block_visitor<Fixup> call_frame_visitor(parent, fixup);
-  parent->iterate_active_callstacks(call_frame_visitor, fixup);
+  std::set<context*>::const_iterator begin = parent->active_contexts.begin();
+  std::set<context*>::const_iterator end = parent->active_contexts.end();
+  while (begin != end) {
+    parent->iterate_callstack(*begin++, call_frame_visitor, fixup);
+  }
 }
 
 template <typename Fixup>

diff --git a/vm/vm.hpp b/vm/vm.hpp
index 15ce00b534726954bd07e6ab326467f56e9dbb62..f98b210c5f3027bd744c94abd71e3680f5fe1e4a 100644 (file)
--- a/vm/vm.hpp
+++ b/vm/vm.hpp
@@ -177,15 +177,6 @@ struct factor_vm {
   void primitive_check_datastack();
   void primitive_load_locals();
 
-  template <typename Iterator, typename Fixup>
-  void iterate_active_callstacks(Iterator& iter, Fixup& fixup) {
-    std::set<context*>::const_iterator begin = active_contexts.begin();
-    std::set<context*>::const_iterator end = active_contexts.end();
-    while (begin != end) {
-      iterate_callstack(*begin++, iter, fixup);
-    }
-  }
-
   // run
   void primitive_exit();
   void primitive_nano_count();