Skip to content

Commit

Permalink
Remove unnecessary lock
Browse files Browse the repository at this point in the history
Signed-off-by: Jorge Bescos Gascon <jorge.bescos.gascon@oracle.com>
  • Loading branch information
jbescos committed Jul 7, 2021
1 parent ea56d57 commit 25792ff
Showing 1 changed file with 4 additions and 5 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2019 Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2021 Oracle and/or its affiliates. All rights reserved.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Public License v. 2.0, which is available at
Expand Down Expand Up @@ -195,12 +195,11 @@ private void handleIllegalStateTransition(HttpConnection.State oldState, HttpCon
throw new IllegalStateException("Illegal state transition, old state: " + oldState + " new state: " + newState);
}

private synchronized void removeAllPendingWithError(Throwable t) {
for (RequestRecord requestRecord : pendingRequests) {
private void removeAllPendingWithError(Throwable t) {
RequestRecord requestRecord = null;
while ((requestRecord = pendingRequests.poll()) != null) {
requestRecord.completionHandler.failed(t);
}

pendingRequests.clear();
}

private class ConnectionStateListener implements HttpConnection.StateChangeListener {
Expand Down

0 comments on commit 25792ff

Please sign in to comment.