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.
STDERR:
-------------
PR: https://git.openjdk.org/jdk/pull/12690
More information about the core-libs-dev
mailing list