Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)` and
`String.getBytes(Charset)` instead of passing the charset name.
-
Commit messages:
- 8305746: InitializeEncoding should cache Charset object instead of charset
name
Changes: https://git.openjdk.org/jdk/p
On Mon, 17 Apr 2023 17:47:04 GMT, Peter Hofer wrote:
> Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)` and
> `String.getBytes(Charset)` instead of passing the charset name.
@naotoj please find here my changes to `InitializeEncoding`.
-
PR Comment: https:/
> Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)` and
> `String.getBytes(Charset)` instead of passing the charset name.
Peter Hofer has updated the pull request with a new target base due to a merge
or a rebase. The pull request now contains one commit:
8305746: Init
On Tue, 18 Apr 2023 11:34:45 GMT, Peter Hofer wrote:
>> Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)`
>> and `String.getBytes(Charset)` instead of passing the charset name.
>
> Peter Hofer has updated the pull request with a new target base due to a
> merge or a reba
On Tue, 18 Apr 2023 21:04:05 GMT, Naoto Sato wrote:
>> Peter Hofer has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains one commit:
>>
>> 8305746: InitializeEncoding should cache Charset object instead of charset
>> name
>
> src/jav
> Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)` and
> `String.getBytes(Charset)` instead of passing the charset name.
Peter Hofer has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brou
On Wed, 19 Apr 2023 07:08:53 GMT, Peter Hofer wrote:
>> Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)`
>> and `String.getBytes(Charset)` instead of passing the charset name.
>
> Peter Hofer has updated the pull request with a new target base due to a
> merge or a reba
On Wed, 19 Apr 2023 06:50:45 GMT, Peter Hofer wrote:
>> src/java.base/share/native/libjava/jni_util.c line 757:
>>
>>> 755: jnuEncoding = (*env)->NewGlobalRef(env, charset.l);
>>> 756: (*env)->DeleteLocalRef(env, charset.l);
>>> 757: break;
>>
>>
On Wed, 19 Apr 2023 07:08:53 GMT, Peter Hofer wrote:
>> Store `Charset` object in `jnuEncoding` and use `String(byte[], Charset)`
>> and `String.getBytes(Charset)` instead of passing the charset name.
>
> Peter Hofer has updated the pull request with a new target base due to a
> merge or a reba
