Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-26 Thread Chen Liang
On Thu, 25 Jul 2024 23:05:46 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-26 Thread Shaojin Wen
On Fri, 26 Jul 2024 12:25:24 GMT, Stephen Colebourne wrote: > This change is fine, however the comments and performance testing assume that > a date is always 10 characters long which isn't true, as the year could be > greater than . The scenario where year is greater than also works

Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-26 Thread Chen Liang
On Thu, 25 Jul 2024 23:05:46 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-26 Thread Stephen Colebourne
On Thu, 25 Jul 2024 23:05:46 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-26 Thread duke
On Thu, 25 Jul 2024 23:05:46 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-25 Thread Chen Liang
On Thu, 25 Jul 2024 23:05:46 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v4]

2024-07-25 Thread Shaojin Wen
> 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 sinc

Re: RFR: 8337168: Optimize LocalDateTime.toString [v3]

2024-07-25 Thread Chen Liang
On Thu, 25 Jul 2024 22:51:09 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v3]

2024-07-25 Thread Shaojin Wen
> 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 sinc

Re: RFR: 8337168: Optimize LocalDateTime.toString [v2]

2024-07-25 Thread Chen Liang
On Thu, 25 Jul 2024 21:52:03 GMT, Shaojin Wen 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 th

Re: RFR: 8337168: Optimize LocalDateTime.toString [v2]

2024-07-25 Thread Shaojin Wen
> 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 two additional commits sin

Re: RFR: 8337168: Optimize LocalDateTime.toString

2024-07-25 Thread Roger Riggs
On Thu, 25 Jul 2024 05:06:17 GMT, Shaojin Wen 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. src/java.base/share/classes/java

Re: RFR: 8337168: Optimize LocalDateTime.toString

2024-07-25 Thread Naoto Sato
On Thu, 25 Jul 2024 05:06:17 GMT, Shaojin Wen 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. LGTM src/java.base/share/classe

Re: RFR: 8337168: Optimize LocalDateTime.toString

2024-07-25 Thread Chen Liang
On Thu, 25 Jul 2024 06:19:46 GMT, Shaojin Wen 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. > > Below are the performance

Re: RFR: 8337168: Optimize LocalDateTime.toString

2024-07-25 Thread duke
On Thu, 25 Jul 2024 05:06:17 GMT, Shaojin Wen 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. @wenshao Your change (at versio

Re: RFR: 8337168: Optimize LocalDateTime.toString

2024-07-25 Thread Chen Liang
On Thu, 25 Jul 2024 05:06:17 GMT, Shaojin Wen 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. Simple and straightforward impro

Re: RFR: 8337168: Optimize LocalDateTime.toString

2024-07-24 Thread Shaojin Wen
On Thu, 25 Jul 2024 05:06:17 GMT, Shaojin Wen 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. Below are the performance number