[refactor] Update GetAddr_() function signature

Update so the internal function signature matches the external one, as is the
case for the other addrman functions.

1 files changed, 3 insertions, 2 deletions

diff --git a/src/addrman_impl.h b/src/addrman_impl.h
index 1d13df803d..918034caf8 100644
--- a/src/addrman_impl.h
+++ b/src/addrman_impl.h
@@ -242,12 +242,13 @@ private:
     /**
      * Return all or many randomly selected addresses, optionally by network.
      *
-     * @param[out] vAddr         Vector of randomly selected addresses from vRandom.
      * @param[in] max_addresses  Maximum number of addresses to return (0 = all).
      * @param[in] max_pct        Maximum percentage of addresses to return (0 = all).
      * @param[in] network        Select only addresses of this network (nullopt = all).
+     *
+     * @returns                  A vector of randomly selected addresses from vRandom.
      */
-    void GetAddr_(std::vector<CAddress>& vAddr, size_t max_addresses, size_t max_pct, std::optional<Network> network) const EXCLUSIVE_LOCKS_REQUIRED(cs);
+    std::vector<CAddress> GetAddr_(size_t max_addresses, size_t max_pct, std::optional<Network> network) const EXCLUSIVE_LOCKS_REQUIRED(cs);
 
     /** We have successfully connected to this peer. Calling this function
      *  updates the CAddress's nTime, which is used in our IsTerrible()