ping
On 12 March 2014 10:48, Stephen Colebourne <scolebou...@joda.org> wrote: > This is a request for review of this bug: > https://bugs.openjdk.java.net/browse/JDK-8035099 > > The implementation for LocalTime with(MILLI_OF_DAY, n) and LocalTime > with(MICRO_OF_DAY, n) fails to match the specification. > > LocalTime base = LocalTime.of(12, 30, 40, 987654321); > > LocalTime result = base.with(MILLI_OF_DAY, 0); > expected: 00:00:00.000000000 > was: 00:00:00.000654321 > > LocalTime result = base.with(MICRO_OF_DAY, 0); > expected: 00:00:00.000000000 > was: 00:00:00.000000321 > > The spec is clear in both cases - "This completely replaces the time > and is equivalent to using {@link #ofNanoOfDay(long)}", thus this is > clearly a bug. > > > Proposed patch: > https://gist.github.com/jodastephen/9057131 > The patch includes no spec changes. > The patch includes new TCK tests that are derived explicitly from the spec. > > I need a reviewer and a committer please. > thanks > Stephen