[refactor] Remove unused ForEachNodeThen() template

Author: jnewbery

src/net.h

@@ -844,28 +844,6 @@ class CConnman
         }
     };
 
-    template<typename Callable, typename CallableAfter>
-    void ForEachNodeThen(Callable&& pre, CallableAfter&& post)
-    {
-        LOCK(cs_vNodes);
-        for (auto&& node : vNodes) {
-            if (NodeFullyConnected(node))
-                pre(node);
-        }
-        post();
-    };
-
-    template<typename Callable, typename CallableAfter>
-    void ForEachNodeThen(Callable&& pre, CallableAfter&& post) const
-    {
-        LOCK(cs_vNodes);
-        for (auto&& node : vNodes) {
-            if (NodeFullyConnected(node))
-                pre(node);
-        }
-        post();
-    };
-
     // Addrman functions
     std::vector<CAddress> GetAddresses(size_t max_addresses, size_t max_pct);
     /**

src/test/fuzz/connman.cpp

@@ -64,9 +64,6 @@ FUZZ_TARGET_INIT(connman, initialize_connman)
             [&] {
                 connman.ForEachNode([](auto) {});
             },
-            [&] {
-                connman.ForEachNodeThen([](auto) {}, []() {});
-            },
             [&] {
                 (void)connman.ForNode(fuzzed_data_provider.ConsumeIntegral<NodeId>(), [&](auto) { return fuzzed_data_provider.ConsumeBool(); });
             },