<i18n dev> Integrated: 8313231: Redundant if statement in ZoneInfoFile

John Jiang jjiang at openjdk.org
Mon Jul 31 07:53:55 UTC 2023


On Thu, 27 Jul 2023 06:46:46 GMT, John Jiang <jjiang at 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 <jjiang at 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


More information about the i18n-dev mailing list