cleanup: remove dead code - SkipSet is not used anywhere

Author: knst

src/Makefile.am

@@ -327,7 +327,6 @@ BITCOIN_CORE_H = \
   util/serfloat.h \
   util/settings.h \
   util/ranges_set.h \
-  util/skip_set.h \
   util/sock.h \
   util/string.h \
   util/time.h \
@@ -746,7 +745,6 @@ libbitcoin_util_a_SOURCES = \
   util/readwritefile.cpp \
   util/settings.cpp \
   util/ranges_set.cpp \
-  util/skip_set.cpp \
   util/spanparsing.cpp \
   util/strencodings.cpp \
   util/time.cpp \

src/test/util_tests.cpp

@@ -16,7 +16,6 @@
 #include <util/message.h> // For MessageSign(), MessageVerify(), MESSAGE_MAGIC
 #include <util/moneystr.h>
 #include <util/ranges_set.h>
-#include <util/skip_set.h>
 #include <util/spanparsing.h>
 #include <util/strencodings.h>
 #include <util/string.h>
@@ -2194,31 +2193,6 @@ BOOST_AUTO_TEST_CASE(test_CRanges)
     }
 }
 
-BOOST_AUTO_TEST_CASE(test_SkipSet)
-{
-    std::mt19937 gen;
-    for (size_t test = 0; test < 17; ++test) {
-        std::uniform_int_distribution<uint64_t> dist_value(0, (1 << test));
-        size_t skip_size = test ? (1 << (test - 1)) : 1;
-        CSkipSet set_1{skip_size};
-        std::unordered_set<uint64_t> set_2;
-        for (size_t iter = 0; iter < (1 << test) * 2; ++iter) {
-            uint64_t value = dist_value(gen);
-            BOOST_CHECK_EQUAL(set_1.Contains(value), !!set_2.count(value));
-            if (!set_1.Contains(value) && set_1.CanBeAdded(value)) {
-                BOOST_CHECK(!set_1.Contains(value));
-                BOOST_CHECK(set_1.Add(value));
-                set_2.insert(value);
-            }
-            BOOST_CHECK_EQUAL(set_1.Contains(value), !!set_2.count(value));
-            BOOST_CHECK_EQUAL(set_1.Size(), set_2.size());
-        }
-        if (test > 4) {
-            BOOST_CHECK(set_1.Size() > ((1 << test) / 4));
-        }
-    }
-}
-
 static std::string SpanToStr(const Span<const char>& span)
 {
     return std::string(span.begin(), span.end());