RFR (S) 8232168: Fix non wide char canonicalization on Windows

David Holmes david.holmes at oracle.com
Wed Oct 16 03:58:32 UTC 2019


Hi Ralf,

This isn't a hotspot issue but a core-libs one.

The use of a hotspot-runtime test seems more opportunistic than anything 
else - is it just for code coverage? I would expect to find a more 
appropriate test somewhere in core-libs.

Thanks,
David

On 16/10/2019 1:19 am, Schmelter, Ralf wrote:
> Hello,
> 
> this is a small change which fixes the some problems with the canonicalize() method in canonicalize_md.c. Currently it lets the wcanonicalize() method do the  work for path lengths > MAX_PATH. This change always calls the wcanonicalize() method, since the old implementation doesn't work for all path lengths <= MAX_PATH. In addition the canonicalizeWithPrefix() method has been removed, since it seems to be not used anymore.
> 
> Bug report: https://bugs.openjdk.java.net/browse/JDK-8232168
> Webrev: http://cr.openjdk.java.net/~rschmelter/webrevs/8232168/webrev.0/
> 
> Best regards,
> Ralf
> 


More information about the hotspot-runtime-dev mailing list