<i18n dev> RFR: 8302983: ZoneRulesProvider.registerProvider() twice will remove provider [v2]

Naoto Sato naoto at openjdk.org
Fri Feb 24 21:55:07 UTC 2023

On Fri, 24 Feb 2023 19:52:59 GMT, Madjosz <duke at openjdk.org> wrote:

>> Fixes JDK-8302983 (and duplicate JDK-8302898)
> Madjosz has updated the pull request incrementally with one additional commit since the last revision:
>   apply review comments

Changes requested by naoto (Reviewer).

test/jdk/java/time/test/java/time/zone/TestZoneRulesProvider.java line 107:

> 105: 
> 106:         // availability check
> 107:         assertTrue(ZoneId.getAvailableZoneIds().contains(zone), "Unexpected non-availability for " + id);

Did you actually run the test??? It still does not compile. Please make sure you run the test under the modified jdk and passes.



PR: https://git.openjdk.org/jdk/pull/12690

More information about the i18n-dev mailing list