From b7e2d3a10bd3b036ca3c4ba19e3b39590e9d7356 Mon Sep 17 00:00:00 2001
From: zzz <zzz@mail.i2p>
Date: Wed, 20 May 2009 18:12:17 +0000
Subject: [PATCH] add some comments
---
.../src/net/i2p/router/peermanager/PeerManager.java | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/router/java/src/net/i2p/router/peermanager/PeerManager.java b/router/java/src/net/i2p/router/peermanager/PeerManager.java
index 146e6a3f9b..7086f1cfc6 100644
--- a/router/java/src/net/i2p/router/peermanager/PeerManager.java
+++ b/router/java/src/net/i2p/router/peermanager/PeerManager.java
@@ -30,6 +30,16 @@ import net.i2p.util.SimpleTimer;
/**
* Manage the current state of the statistics
*
+ * All the capabilities methods appear to be almost unused -
+ * TunnelPeerSelector just looks for unreachables, and that's it?
+ * If so, a lot of this can go away, including the array of 26 ArrayLists,
+ * and a lot of synchronization on _capabilitiesByPeer.
+ *
+ * We don't trust any published capabilities except for 'K' and 'U'.
+ * This should be cleaned up.
+ *
+ * setCapabilities() and removeCapabilities() can just add/remove the profile and that's it.
+ *
*/
class PeerManager {
private Log _log;
--
GitLab