RFR: 8234185: Cleanup usage of canonicalize function between libjava, hotspot and libinstrument
Alan Bateman
Alan.Bateman at oracle.com
Thu Nov 21 08:50:31 UTC 2019
On 14/11/2019 15:37, Langer, Christoph wrote:
> Hi,
>
> please review this cleanup change regarding function "canonicalize" of libjava.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8234185
> Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8234185.0/
>
>
> The goal is to cleanup how this function is defined and used. One thing is, that there was an unnecessary wrapper function "Canonicalize" in jni_util.c. It wrapped the call to "canonicalize". We can get rid of this wrapper. Unfortunately, it is not possible to just export "canonicalize" since this will conflict with a method signature from the math library, at least on modern Linuxes. So I decided to call the method JDK_Canonicalize and will correctly define it in jdk_util.h which can be included everywhere.
>
I think this change is okay. My main concern when initially seeing this
go by was that it would leak the \\?\ or \\?\UNC\ prefix into the
canonical File when it wasn't there previously, this would of course
have several implications. But I think you have it right and this is, as
you position, just refactoring/cleanup.
-Alan
More information about the hotspot-runtime-dev
mailing list