On Thu, 10 Mar 2022 06:52:14 GMT, John Jiang <jji...@openjdk.org> wrote:

> In class sun.security.ssl.ProtocolVersion, the exception message for 
> unsupported protocol needs a space.
> 
> ProtocolVersion pv = ProtocolVersion.nameOf(pn);
> if (pv == null) {
>     throw new IllegalArgumentException(
>            "Unsupported protocol" + pn);
> }

Changes requested by mullan (Reviewer).

src/java.base/share/classes/sun/security/ssl/ProtocolVersion.java line 293:

> 291:             if (pv == null) {
> 292:                 throw new IllegalArgumentException(
> 293:                         "Unsupported protocol " + pn);

I think you should also add a colon so this reads better, i.e.:  `"Unsupported 
protocol: " + pn);`

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

PR: https://git.openjdk.java.net/jdk/pull/7769

Reply via email to