> > The change looks good to me. But I would say the assertion in
> CalendarDataUtility in line 266 is pointless now, isn't it?
> 
> Yes, but would not hurt leaving it. It would catch error if yet another
> case is installed (and it forgot to provide a default value) in the
> switch statement. So I just left it.

Ok, sounds reasonable.

What's your target for pushing this? I'd like to backport it to OpenJDK 11.0.4 
to fix the regression there...

/Christoph

Reply via email to