On Fri, 19 Jul 2024 09:52:50 GMT, Andrey Turbanov <aturba...@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 implementations, in the subclasses too, were there to match the similar methods in LocalDate. At one point in the API development they were visible and had been included in the ChronoLocalDate interface. Simplification prevailed. 7-day weeks are ubiquitous in the supported calendars but since no-one has missed these methods they are unneeded. I agree with Stephen in retaining plusWeeks and its use in the implementation of plus(long, unit). ------------- PR Comment: https://git.openjdk.org/jdk/pull/20250#issuecomment-2273493709
