if (i < savingsInstantTransitions.length) { // javazic writes the last GMT offset into index 0! if (i < savingsInstantTransitions.length) { offsets[0] = standardOffsets[standardOffsets.length - 1] * 1000; nOffsets = 1; } ... }
The second if statement looks unnecessary. ------------- Commit messages: - 8313231: Redundant if statement in ZoneInfoFile Changes: https://git.openjdk.org/jdk/pull/15052/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15052&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8313231 Stats: 4 lines in 1 file changed: 0 ins; 1 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/15052.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/15052/head:pull/15052 PR: https://git.openjdk.org/jdk/pull/15052