> > 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