RFR: 8337168: Optimize LocalDateTime.toString [v4]
Chen Liang
liach at openjdk.org
Fri Jul 26 14:41:38 UTC 2024
On Thu, 25 Jul 2024 23:05:46 GMT, Shaojin Wen <duke at openjdk.org> wrote:
>> The current LocalDateTime.toString implementation concatenates the toString results of date and time, which can be passed to StringBuilder to reduce additional object allocation and improve performance.
>
> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
>
> Update src/java.base/share/classes/java/time/LocalTime.java
>
> Co-authored-by: Chen Liang <liach at openjdk.org>
The logic is fine; it's that the comment is inaccurate, that there may be StringBuilder reallocations if the year is >9999 or <0.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20321#issuecomment-2252911244
More information about the core-libs-dev
mailing list