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