On Thu, 27 Jul 2023 06:46:46 GMT, John Jiang <jji...@openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: f8c2b7fe
Author:    John Jiang <jji...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/f8c2b7fee101d66107704b3ee464737c5ccdc13a
Stats:     4 lines in 1 file changed: 0 ins; 1 del; 3 mod

8313231: Redundant if statement in ZoneInfoFile

Reviewed-by: jiefu, scolebourne

-------------

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

Reply via email to