Thanks Chris!

I was going to answer that this was not possible as it requires
multiple return values but I found a way:

http://cr.openjdk.java.net/~dfuchs/webrev_8239052/webrev.02/

best regards,

-- daniel

On 22/02/2020 13:53, Chris Hegarty wrote:
On 21 Feb 2020, at 17:41, Daniel Fuchs<daniel.fu...@oracle.com>  wrote:

Thanks John!

I have verified that indeed TLSv1.3 is selected by default,
and updated the fix as per your suggestion:

http://cr.openjdk.java.net/~dfuchs/webrev_8239052/webrev.01/
Good sleuthing Daniel.

Your changes look good, just wondering if they could be factored out
into a separate method, say getLong(ByteBuffer), so that the crux of the
test logic in onNext would not be slightly obfuscated by this?

-Chris.

Reply via email to