RFR: 8316190: Improve MemorySegment::toString
Claes Redestad
redestad at openjdk.org
Thu Sep 14 14:26:41 UTC 2023
On Thu, 14 Sep 2023 13:44:59 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> src/java.base/share/classes/jdk/internal/foreign/Utils.java line 277:
>>
>>> 275: }
>>> 276:
>>> 277: public static String toHexString(long value) {
>>
>> We intend to use this utility method in coming PRs.
>
> The String concat code is really good at optimizing, but this will require a new string be created.
> $0.02,Roger
Why not `HexFormat.of().withPrefix("0x").formatHexString(long)` I thought, then realized there's no such method, only `toHexDigits` which ignores the `HexFormat.prefix`. Having a format method that adds the prefix (and any suffix + delimiters) means creating this string could be done in a single step without concatenation, and we'd consolidate all manners of usage to a single place. Seems there might be room for improvement here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15740#discussion_r1326032912
More information about the core-libs-dev
mailing list