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

2025-02-07 Thread duke
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: >> >>

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

2025-02-07 Thread Bradford Wetmore
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. This pull request has now been integrated. Changeset: 88a84835 Author:Bradford Wetmore URL: http

RFR: 8349662: SSLTube SSLSubscriptionWrapper has potential races when switching subscriptions

2025-02-07 Thread Daniel Fuchs
Hi, Please find here a change that fixes a potential race condition in SSLTube.SSLSubscriptionWrapper. Typically the race may get triggered if the demand increased by request() is not exhausted by the time the subscription is switched by setSubscription. Some synchronization is required to p

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

2025-02-07 Thread Jaikiran Pai
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 [v4]

2025-02-07 Thread Jamil Nimeh
On Thu, 6 Feb 2025 19:31:24 GMT, Bradford Wetmore wrote: >> 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 las