On Wed, 22 Sep 2021 21:05:34 GMT, Mahendra Chhipa
<[email protected]> wrote:
>> Dependencies of TestHttpServre, HttpTransaction, HttpCallback are removed
>> from following tests:
>> open/test/jdk/sun/net/www/protocol/http/ResponseCacheStream.java
>> open/test/jdk/sun/net/www/protocol/http/SetChunkedStreamingMode.java
>> open/test/jdk/sun/net/www/protocol/http/RelativeRedirect.java
>> open/test/jdk/sun/net/www/protocol/http/B6296310.java
>
> Mahendra Chhipa has updated the pull request with a new target base due to a
> merge or a rebase. The incremental webrev excludes the unrelated changes
> brought in by the merge/rebase. The pull request contains three additional
> commits since the last revision:
>
> - Updated execution mode to othervm to resolve cleaning up thread issue.
> - Merge branch 'master' into JDK-8273142
> - JDK-8273142 : Remove dependancy of TestHttpServer, HttpTransaction,
> HttpCallback from open/test/jdk/sun/net/www/protocol/http/ tests
Changes requested by michaelm (Reviewer).
test/jdk/sun/net/www/protocol/http/B6296310.java line 103:
> 101: try(PrintWriter pw = new PrintWriter(trans.getResponseBody())) {
> 102: pw.print("");
> 103: }
Not sure what the purpose of this block is, which apparently writes no bytes to
the OutputStream?
-------------
PR: https://git.openjdk.java.net/jdk/pull/5352
