stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize

Browse Source 
Reviewed-by: dfuchs
This commit is contained in:
Michael McMahon 2021-08-30 08:57:17 +00:00
parent f55d5ab517
commit 16e83058ca
2 changed files with 2 additions and 6 deletions
src/java.net.http/share/classes/jdk/internal/net/http
Http2ClientImpl.javaResponseContent.java

3
src/java.net.http/share/classes/jdk/internal/net/http/Http2ClientImpl.java

@ -237,8 +237,7 @@ class Http2ClientImpl {
// The default is the max between the stream window size
// and the connection window size.
int defaultValue = Math.min(Integer.MAX_VALUE,
Math.max(streamWindow, K*K*32));
int defaultValue = Math.max(streamWindow, K*K*32);
return getParameter(
"jdk.httpclient.connectionWindowSize",

5
src/java.net.http/share/classes/jdk/internal/net/http/ResponseContent.java

@ -489,7 +489,6 @@ class ResponseContent {
debug.log("already closed: " + closedExceptionally);
return;
}
boolean completed = false;
try {
if (debug.on())
debug.log("Parser got %d bytes ", b.remaining());
@ -506,9 +505,7 @@ class ResponseContent {
} catch (Throwable t) {
if (debug.on()) debug.log("Unexpected exception", t);
closedExceptionally = t;
if (!completed) {
onComplete.accept(t);
}
onComplete.accept(t);
}
}