RFR: JDK-8068803:Performance of LocalDate.plusDays could be better

Roger Riggs Roger.Riggs at Oracle.com
Tue Jan 5 18:41:36 UTC 2016


Hi Nadeesh,

LocalDate.java: +1374:
   For the most common case of dom > 0 && <= 28, I would have explicitly 
and immediately returned the new LocalDate.

if (dom > 0 && dom <= 28) {  return LocalDate.of(year, month, (int) 
dom); } ...


TCKLocalDate.java:
  - Since the test_plusDays_normal is being replaced, its test case 
should be included in the DataProvider

{LocalDate.of(2007, 7, 15), 1, LocalDate.of(2007, 7, 16)}

Thanks, Roger

On 1/5/2016 1:05 PM, nadeesh tv wrote:
> Hi all,
>
> Please review  a  fix for 
> https://bugs.openjdk.java.net/browse/JDK-8068803
>  web rev :   http://cr.openjdk.java.net/~ntv/8068803/webrev.00/
>
> Special thanks for Stephen for  providing the source code patch
>




More information about the core-libs-dev mailing list