RFR JDK-8191533,jar --describe-module prints service provider class names in lower case
Alan Bateman
Alan.Bateman at oracle.com
Wed May 16 07:19:27 UTC 2018
On 16/05/2018 00:51, Xueming Shen wrote:
> Hi,
>
> Please help review the change for JDK-8191533
>
> issue: https://bugs.openjdk.java.net/browse/JDK-8191533
> webrev: http://cr.openjdk.java.net/~sherman/8191533/webrev/
>
> The jmod change has been verify by applying jmod on those modules at
> jdk home.
This looks okay but I think it could be improved if you have
toLowerCaseString and toString rather than toStringWithCase and
toString. Also they can both drop the isEmpty check and prefixing the
string with " ". Instead I think the usages should be fixed to add the
trailing space that they are missing, e.g. " with" to " with ", " to" to
" to " ....
-Alan
More information about the core-libs-dev
mailing list