From e3103762b6b440acfcd67f371da163ada48e0128 Mon Sep 17 00:00:00 2001
From: dev <dev@robertfoss.se>
Date: Mon, 6 Apr 2015 15:00:40 +0000
Subject: [PATCH] findbugs: Removed synchronization on concurrent object
---
.../net/i2p/router/transport/udp/EstablishmentManager.java | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/router/java/src/net/i2p/router/transport/udp/EstablishmentManager.java b/router/java/src/net/i2p/router/transport/udp/EstablishmentManager.java
index c44339dfe6..9eecec7618 100644
--- a/router/java/src/net/i2p/router/transport/udp/EstablishmentManager.java
+++ b/router/java/src/net/i2p/router/transport/udp/EstablishmentManager.java
@@ -595,12 +595,7 @@ class EstablishmentManager {
}
if (_outboundStates.size() < getMaxConcurrentEstablish() && !_queuedOutbound.isEmpty()) {
- // in theory shouldn't need locking, but
- // getting IllegalStateExceptions on old Java 5,
- // which hoses this state.
- synchronized(_queuedOutbound) {
- locked_admitQueued();
- }
+ locked_admitQueued();
}
//remaining = _queuedOutbound.size();
--
GitLab