Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v6]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 14:26:23 GMT, Volkan Yazici wrote: >> Adds `test.lib.Utils::createTempFileOfSize` to generate >> `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory >> contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file >> tracked by git: >> >>

Re: RFR: 8349121: SSLParameters.setApplicationProtocols() ALPN example could be clarified [v3]

2025-02-04 Thread Bradford Wetmore
On Tue, 4 Feb 2025 17:18:09 GMT, Daniel Fuchs wrote: >> Bradford Wetmore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Codereview Comments > > src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 668: > >> 666: *

Re: RFR: 8349121: SSLParameters.setApplicationProtocols() ALPN example could be clarified [v3]

2025-02-04 Thread Bradford Wetmore
> Update and clarify the sample code. > > Docs only, no additional testing other than verifying javadoc is correctly > output. Bradford Wetmore has updated the pull request incrementally with one additional commit since the last revision: Codereview Comments - Changes: - all:

Re: RFR: 8349121: SSLParameters.setApplicationProtocols() ALPN example could be clarified [v2]

2025-02-04 Thread Bradford Wetmore
> Update and clarify the sample code. > > Docs only, no additional testing other than verifying javadoc is correctly > output. Bradford Wetmore 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 th

Re: RFR: 8349121: SSLParameters.setApplicationProtocols() ALPN example could be clarified

2025-02-04 Thread Daniel Fuchs
On Fri, 31 Jan 2025 01:45:47 GMT, Bradford Wetmore wrote: > Update and clarify the sample code. > > Docs only, no additional testing other than verifying javadoc is correctly > output. src/java.base/share/classes/javax/net/ssl/SSLParameters.java line 668: > 666: * String encodedHukUn

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v6]

2025-02-04 Thread Daniel Fuchs
On Tue, 4 Feb 2025 14:26:23 GMT, Volkan Yazici wrote: >> Adds `test.lib.Utils::createTempFileOfSize` to generate >> `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory >> contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file >> tracked by git: >> >>

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v4]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 10:47:22 GMT, Daniel Fuchs wrote: >> Volkan Yazici has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Assert that multiple files can be served using the same >> `FileServerHandler` >> - Remove redundant `@build` depen

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v5]

2025-02-04 Thread Daniel Fuchs
On Tue, 4 Feb 2025 12:04:47 GMT, Volkan Yazici wrote: >> Adds `test.lib.Utils::createTempFileOfSize` to generate >> `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory >> contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file >> tracked by git: >> >>

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v6]

2025-02-04 Thread Volkan Yazici
> Adds `test.lib.Utils::createTempFileOfSize` to generate > `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory > contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file > tracked by git: > > > $ git ls-files | while read f; do echo -e $(stat -c %s "$f

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v5]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 14:15:00 GMT, Daniel Fuchs wrote: >> Volkan Yazici has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Replace `assert`s with conditionally thrown exceptions > > test/jdk/java/net/httpclient/http2/FixedThreadPoolTest.java

Integrated: 8349135: Add tests for HttpRequest.Builder.copy()

2025-02-04 Thread Volkan Yazici
On Mon, 3 Feb 2025 10:26:59 GMT, Volkan Yazici wrote: > Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. This pull request has now been integrated. Changeset: d699aba2 Author:Volkan Yazici URL: https://git.openjdk.org/jdk/commit/d699aba26f65ce1941bfbaaf1918c987dd0

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v5]

2025-02-04 Thread duke
On Tue, 4 Feb 2025 12:22:28 GMT, Volkan Yazici wrote: >> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Fix `assertEquals()` argument order @vy Your ch

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v5]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 12:22:28 GMT, Volkan Yazici wrote: >> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Fix `assertEquals()` argument order Attached suc

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v5]

2025-02-04 Thread Jaikiran Pai
On Tue, 4 Feb 2025 12:22:28 GMT, Volkan Yazici wrote: >> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Fix `assertEquals()` argument order Thank you fo

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v4]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 11:22:41 GMT, Jaikiran Pai wrote: >> Volkan Yazici has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Replace `assert`s with explicit checks throwing `AssertionError`s > > test/jdk/java/net/httpclient/HttpRequestBuilderTe

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v5]

2025-02-04 Thread Volkan Yazici
> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision: Fix `assertEquals()` argument order - Changes: - all: https://git.openjdk.org/jdk/pull/23415/files

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v4]

2025-02-04 Thread Daniel Fuchs
On Tue, 4 Feb 2025 11:07:09 GMT, Volkan Yazici wrote: >> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Replace `assert`s with explicit checks throwing `

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v4]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 10:46:33 GMT, Daniel Fuchs wrote: >> Volkan Yazici has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Assert that multiple files can be served using the same >> `FileServerHandler` >> - Remove redundant `@build` depen

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v5]

2025-02-04 Thread Volkan Yazici
> Adds `test.lib.Utils::createTempFileOfSize` to generate > `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory > contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file > tracked by git: > > > $ git ls-files | while read f; do echo -e $(stat -c %s "$f

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v4]

2025-02-04 Thread Jaikiran Pai
On Tue, 4 Feb 2025 11:07:09 GMT, Volkan Yazici wrote: >> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. > > Volkan Yazici has updated the pull request incrementally with one additional > commit since the last revision: > > Replace `assert`s with explicit checks throwing `

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v4]

2025-02-04 Thread Volkan Yazici
> Adds `HttpRequest.Builder::copy` tests to `HttpRequestBuilderTest`. Volkan Yazici has updated the pull request incrementally with one additional commit since the last revision: Replace `assert`s with explicit checks throwing `AssertionError`s - Changes: - all: https://git.ope

Re: RFR: 8349135: Add tests for HttpRequest.Builder.copy() [v3]

2025-02-04 Thread Volkan Yazici
On Tue, 4 Feb 2025 07:24:20 GMT, Jaikiran Pai wrote: >> Volkan Yazici 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 >> commit

Re: RFR: 8343074: test/jdk/com/sun/net/httpserver/docs/test1/largefile.txt could be generated [v4]

2025-02-04 Thread Daniel Fuchs
On Mon, 3 Feb 2025 19:26:03 GMT, Volkan Yazici wrote: >> Adds `test.lib.Utils::createTempFileOfSize` to generate >> `test/jdk/com/sun/net/httpserver/docs` contents at runtime. This directory >> contains `largefile.txt` of size 2.6MiB showing up as the 4th largest file >> tracked by git: >> >>