Integrated: 8297311: Improve exception message thrown by java.net.HostPortrange::toLowerCase(String s)

Jaikiran Pai jpai at openjdk.org
Wed Nov 30 01:14:27 UTC 2022


On Tue, 29 Nov 2022 05:48:26 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this change which proposes to enhance an exception message in `java.net.HostPortrange::toLowerCase`, as requested in https://bugs.openjdk.org/browse/JDK-8297311?
> 
> The commit here includes (only) the character, which was found to be invalid, in the exception message. The character will be printed out in `\uXXXX` format to take into account any unprintable characters.
> 
> A new test has been introduced to verify that the introduction of this change doesn't introduce any regression and does indeed include the expected exception message.

This pull request has now been integrated.

Changeset: 9ced2ea0
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9ced2ea0abdc0646a23873ab8ca443b6ca3e2e0a
Stats:     59 lines in 2 files changed: 58 ins; 0 del; 1 mod

8297311: Improve exception message thrown by java.net.HostPortrange::toLowerCase(String s)

Reviewed-by: dfuchs, michaelm, rriggs

-------------

PR: https://git.openjdk.org/jdk/pull/11407


More information about the net-dev mailing list