From 6d3096eb03079ffe80ab795db0c731f1c336f6b6 Mon Sep 17 00:00:00 2001
From: Boog900 <54e72d8a-345f-4599-bd90-c6b9bc7d0ec5@aleeas.com>
Date: Mon, 13 May 2024 23:56:00 +0100
Subject: [PATCH] small changes
---
p2p/cuprate-p2p/src/client_pool.rs | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/p2p/cuprate-p2p/src/client_pool.rs b/p2p/cuprate-p2p/src/client_pool.rs
index 59d96d30..3b21c790 100644
--- a/p2p/cuprate-p2p/src/client_pool.rs
+++ b/p2p/cuprate-p2p/src/client_pool.rs
@@ -17,6 +17,7 @@ use tokio::sync::mpsc;
use monero_p2p::{
client::{Client, InternalPeerID},
+ handles::ConnectionHandle,
ConnectionDirection, NetworkZone,
};
@@ -24,7 +25,6 @@ mod disconnect_monitor;
mod drop_guard_client;
pub use drop_guard_client::ClientPoolDropGuard;
-use monero_p2p::handles::ConnectionHandle;
/// The client pool, which holds currently connected free peers.
///
@@ -33,8 +33,9 @@ pub struct ClientPool<N: NetworkZone> {
/// The connected [`Client`]s.
clients: DashMap<InternalPeerID<N::Addr>, Client<N>>,
/// A set of outbound clients, as these allow accesses/ mutation from different threads
- /// a peer ID in here does not mean the peer is definitely in `clients` , if the peer is
- /// in both here and `clients` it is defiantly an outbound peer,
+ /// a peer ID in here does not mean the peer is in `clients` as it could have been removed
+ /// by another thread. However, if the peer is in both here and `clients` it is defiantly
+ /// an outbound peer,
outbound_clients: DashSet<InternalPeerID<N::Addr>>,
/// A channel to send new peer ids down to monitor for disconnect.