RFR JDK-8232724: Remove indirection with calling JNU_NewStringPlatform

David Holmes david.holmes at oracle.com
Tue Oct 29 07:49:07 UTC 2019

Hi Alexey,

On 29/10/2019 4:46 am, Alexey Ivanov wrote:
> Hello,
> Please review the following fix which removes indirection in calling 
> JNU_NewStringPlatform via NewStringPlatform.
> bug: https://bugs.openjdk.java.net/browse/JDK-8232724
> webrev: http://cr.openjdk.java.net/~aivanov/8232724/webrev.00/
> It is a follow-up fix to JDK-8232624 and JDK-8231355. It removes 
> NewStringPlatform alias as Claes did in JDK-8231355. To call 
> JNU_NewStringPlatform, the function pointer to_java_string_fn_t is 
> declared with JNICALL (__stdcall); the function is looked up by both the 
> undecorated name and the __stdcall-decorated name for Win32.

That seems fine.

> Tier 1 and 2 tests are all green.
> There are two declarations of the to_java_string_fn_t function pointer: 
> in jvm.h and locally in javaClasses. I searched the source code for it 
> and didn't find any usages but in 
> java_lang_String::create_from_platform_dependent_str. The declaration in 
> jvm.h uses "char *" whereas javaClasses uses "const char *".
> Shall I remove one of them? The one in jvm.h because it's unused?

Yes please remove the unused one in jvm.h.


> Thank you in advance.
> https://bugs.openjdk.java.net/browse/JDK-8232624
> https://bugs.openjdk.java.net/browse/JDK-8231355

More information about the hotspot-runtime-dev mailing list