addrman, refactor: combine two size functions

The functionality of the old size() is covered by the new Size()
when no arguments are specified, so this does not change behavior.

Co-authored-by: Martin Zumsande <mzumsande@gmail.com>

1 files changed, 0 insertions, 11 deletions

diff --git a/src/addrman.cpp b/src/addrman.cpp
index 4ffb7617b0..01ea723dd2 100644
--- a/src/addrman.cpp
+++ b/src/addrman.cpp
@@ -1112,12 +1112,6 @@ int AddrManImpl::CheckAddrman() const
     return 0;
 }
 
-size_t AddrManImpl::size() const
-{
-    LOCK(cs); // TODO: Cache this in an atomic to avoid this overhead
-    return vRandom.size();
-}
-
 size_t AddrManImpl::Size(std::optional<Network> net, std::optional<bool> in_new) const
 {
     LOCK(cs);
@@ -1239,11 +1233,6 @@ template void AddrMan::Unserialize(CHashVerifier<CAutoFile>& s);
 template void AddrMan::Unserialize(CDataStream& s);
 template void AddrMan::Unserialize(CHashVerifier<CDataStream>& s);
 
-size_t AddrMan::size() const
-{
-    return m_impl->size();
-}
-
 size_t AddrMan::Size(std::optional<Network> net, std::optional<bool> in_new) const
 {
     return m_impl->Size(net, in_new);