Re: RFR: 8358170: Repurpose testCompat in test/jdk/java/util/TimeZone/Bug8167143.java [v2]

2025-06-03 Thread Naoto Sato
On Tue, 3 Jun 2025 17:52:13 GMT, Justin Lu wrote: >> Please review this PR which modifies the >> test/jdk/java/util/TimeZone/Bug8167143.java "testCompat" case to be >> repurposed to check the implicit locales for the FALLBACK BreakIterator and >> Collator providers. (FALLBACK is the primary pr

Re: RFR: 8358170: Repurpose testCompat in test/jdk/java/util/TimeZone/Bug8167143.java [v2]

2025-06-03 Thread Justin Lu
On Tue, 3 Jun 2025 17:40:19 GMT, Naoto Sato wrote: >> Justin Lu has updated the pull request incrementally with one additional >> commit since the last revision: >> >> review - var rename > > test/jdk/java/util/TimeZone/Bug8167143.java line 239: > >> 237: */ >> 238: private static v

Re: RFR: 8358170: Repurpose testCompat in test/jdk/java/util/TimeZone/Bug8167143.java [v2]

2025-06-03 Thread Naoto Sato
On Tue, 3 Jun 2025 17:46:11 GMT, Justin Lu wrote: >> Please review this PR which modifies the >> test/jdk/java/util/TimeZone/Bug8167143.java "testCompat" case to be >> repurposed to check the implicit locales for the FALLBACK BreakIterator and >> Collator providers. (FALLBACK is the primary pr

Re: RFR: 8358170: Repurpose testCompat in test/jdk/java/util/TimeZone/Bug8167143.java [v2]

2025-06-03 Thread Justin Lu
> Please review this PR which modifies the > test/jdk/java/util/TimeZone/Bug8167143.java "testCompat" case to be > repurposed to check the implicit locales for the FALLBACK BreakIterator and > Collator providers. (FALLBACK is the primary provider in those cases.) Since > the method is corrected

RFR: 8358170: Repurpose testCompat in test/jdk/java/util/TimeZone/Bug8167143.java

2025-06-03 Thread Justin Lu
Please review this PR which modifies the test/jdk/java/util/TimeZone/Bug8167143.java "testCompat" case to be repurposed to check the implicit locales for the FALLBACK BreakIterator and Collator providers. (FALLBACK is the primary provider in those cases.) Since the method is corrected, it can b