On Mon, 7 Jun 2021 18:46:11 GMT, Naoto Sato wrote:
>> @mkartashev thank you for the detailed explanation.
>>
>> It is not clear to me that the JDK's conformance to being a Unicode
>> application has significantly changed since the evaluation of JDK-8017274 -
>> @naotoj can you comment on tha
On Sun, 6 Jun 2021 22:25:44 GMT, David Holmes wrote:
>> I think we need to establish some common ground before proceeding further
>> with this fix. It's a bit of a long read; please, bear with me.
>>
>> The path name starts its life as a `jstring` in
>> `Java_jdk_internal_loader_NativeLibrarie
On Sun, 6 Jun 2021 22:25:44 GMT, David Holmes wrote:
>> I think we need to establish some common ground before proceeding further
>> with this fix. It's a bit of a long read; please, bear with me.
>>
>> The path name starts its life as a `jstring` in
>> `Java_jdk_internal_loader_NativeLibrarie
On Fri, 4 Jun 2021 14:00:25 GMT, Maxim Kartashev
wrote:
>> Not an expert by my understanding is that the VM only deals with modified
>> UTF-8, as does JNI. So the incoming string should be modified-UTF8 IMO and
>> then converted to UTF16.
>>
>> That said, this is shared code being modified on
On Fri, 4 Jun 2021 14:00:25 GMT, Maxim Kartashev
wrote:
>> Not an expert by my understanding is that the VM only deals with modified
>> UTF-8, as does JNI. So the incoming string should be modified-UTF8 IMO and
>> then converted to UTF16.
>>
>> That said, this is shared code being modified on
On Fri, 4 Jun 2021 02:12:42 GMT, David Holmes wrote:
>> I am not sure we can pass non `modified UTF-8` through `JVM_LoadLibrary()`.
>> Probably some VM folks can enlighten here?
>
> Not an expert by my understanding is that the VM only deals with modified
> UTF-8, as does JNI. So the incoming s
On Thu, 3 Jun 2021 17:51:54 GMT, Naoto Sato wrote:
> I think we can simply limit the test platform only to Windows in @requires
> tag in the test. Also, I would see the test case using some supplementary
> characters.
Done.
-
PR: https://git.openjdk.java.net/jdk/pull/4169
On Thu, 3 Jun 2021 17:48:59 GMT, Naoto Sato wrote:
>> Right; I changed the code in NativeLibraries.c to pass down true UTF-8
>> instead of "modified UTF-8". Please, take a look.
>
> I am not sure we can pass non `modified UTF-8` through `JVM_LoadLibrary()`.
> Probably some VM folks can enlighte
On Thu, 3 Jun 2021 06:59:01 GMT, Maxim Kartashev
wrote:
>> test/hotspot/jtreg/runtime/jni/loadLibraryUnicode/LoadLibraryUnicodeTest.java
>> line 42:
>>
>>> 40: String nativePathSetting = "-Dtest.nativepath=" +
>>> getSystemProperty("test.nativepath");
>>> 41: ProcessBuilder pb
On Thu, 3 Jun 2021 06:55:26 GMT, Maxim Kartashev
wrote:
>> src/hotspot/os/windows/os_windows.cpp line 1491:
>>
>>> 1489: static errno_t convert_UTF8_to_UTF16(char const* utf8_str, LPWSTR*
>>> utf16_str) {
>>> 1490: return convert_to_UTF16(utf8_str, CP_UTF8, utf16_str);
>>> 1491: }
>>
>> IIU
On Tue, 1 Jun 2021 18:42:34 GMT, Naoto Sato wrote:
>> Maxim Kartashev has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Coding style-related corrections.
>> - Corrected the test to use Platform.sharedLibraryExt()
>
> test/hotspot/jtreg/
On Tue, 1 Jun 2021 18:24:05 GMT, Naoto Sato wrote:
>> Maxim Kartashev has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Coding style-related corrections.
>> - Corrected the test to use Platform.sharedLibraryExt()
>
> src/hotspot/os/wind
On Thu, 27 May 2021 16:14:38 GMT, Maxim Kartashev
wrote:
>> Character strings within JVM are produced and consumed in several formats.
>> Strings come from/to Java in the UTF8 format and POSIX APIs (like fprintf()
>> or dlopen()) consume strings also in UTF8. On Windows, however, the
>> situa
> Character strings within JVM are produced and consumed in several formats.
> Strings come from/to Java in the UTF8 format and POSIX APIs (like fprintf()
> or dlopen()) consume strings also in UTF8. On Windows, however, the situation
> is far less simple: some new(er) APIs expect UTF16 (wide-ch
14 matches
Mail list logo