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
