The change seems simple enough. Does the test prove the change though? I thought the issue was with months, not years? Stephen
On Thu, 20 Sep 2018 at 22:32, Naoto Sato <naoto.s...@oracle.com> wrote: > > Hello, > > Please review the fix to the following issue: > > https://bugs.openjdk.java.net/browse/JDK-8210633 > > The proposed changeset is located at: > > http://cr.openjdk.java.net/~naoto/8210633/webrev.00/ > > As the submitter pointed out, the overload method that takes Chronology > needs to be overridden in the anonymous DateTimeTextProvider class. > > Naoto