RFR: 8337838: Remove unused methods from ChronoLocalDateImpl

Andrey Turbanov aturbanov at openjdk.org
Wed Aug 7 05:57:34 UTC 2024


On Tue, 6 Aug 2024 19:46:02 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> A few methods in `java.time.chrono.ChronoLocalDateImpl` are unused and could be removed:
>> 1. plusWeeks(long)
>> 2. minusYears(long)
>> 3. minusMonths(long)
>> 4. minusWeeks(long)
>> 5. minusDays(long)
>> 
>> Tested `test/jdk/java/time` on Linux x64 release
>
> The methods should not be removed, they provide default implementation, even though subclasses define overridden implementations.
> Please close this PR.

@RogerRiggs hm. I removed methods in subclases too - they are unused as well.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/20250#issuecomment-2272673688


More information about the core-libs-dev mailing list