On Fri, 7 Aug 2026 15:00:52 GMT, Oumaiyma Intissar <[email protected]> wrote:
>> Overall looks fine to me. > > Thanks for the review @naotoj. I addressed the comments in the latest update: > - Added CLDR v48.2 context and renamed the explicit DST offset map in > `TestZoneTextPrinterParser.java`. > - Added a CLDR v48.2 comment in `Bug6329116.java` and removed the redundant > US locale check. > - Updated the Morocco comments in `NegativeDSTTest.java` and > `TestZoneRules.java` to clarify that Morocco returns to standard UTC > permanently later in 2026, rather than permanent DST. > - Added back one 2038 Morocco test case with the new UTC/no-DST expectation. > > I reran the focused tests for the changed files and they pass. > Thanks, @OumaIntissar, the update looks good to me. My previous comment about > Morocco moving to permanent DST has been correctly addressed by switching to > UTC, which looks good. I'm Moroccan ;) ------------- PR Comment: https://git.openjdk.org/jdk/pull/32240#issuecomment-5221108606
