RFR: 8310843: Reimplement ByteArray and ByteArrayLittleEndian with Unsafe [v10]
Chen Liang
liach at openjdk.org
Thu Jul 20 10:33:45 UTC 2023
On Thu, 20 Jul 2023 09:39:09 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Actually, a byte buffer is big endian, so some extra code would be required. But maybe that's another helper function:
>>
>>
>> @ForceInline
>> ByteBuffer asBuffer(byte[] array) { return ByteBuffer.wrap(array).order(ByteOrder.nativeOrder()); }
>>
>>
>> And then replace:
>>
>>
>> ByteArray.getChar(array, 42)
>>
>> With
>>
>> asBuffer(array).getChar(42);
>
> Also... in a lot of cases where ByteArray is used (DataXYZStream, ObjectXYZStream) the array being used is a field in the class. So the byte buffer creation can definitively be amortized (or the code changed to work on buffers instead of arrays).
The Unsafe-based writing will be used by `Integer.toString` and `Long.toString` as well; in those cases, will creating a ByteBuffer wrapper be overkill?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14636#discussion_r1269265267
More information about the core-libs-dev
mailing list