From 69bb199a1b37cc9bdce7a496e6722cb5ac02d3e3 Mon Sep 17 00:00:00 2001
From: Peter Edberg <pedberg@unicode.org>
Date: Fri, 30 Sep 2011 21:32:41 +0000
Subject: [PATCH] ICU-8753 Replace confusing use of dst variable with
 prevOffset / newOffset

X-SVN-Rev: 30772
---
 .../core/src/com/ibm/icu/util/Calendar.java   | 22 +++++++++----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/icu4j/main/classes/core/src/com/ibm/icu/util/Calendar.java b/icu4j/main/classes/core/src/com/ibm/icu/util/Calendar.java
index 55b170b5706..9fd3413c57d 100644
--- a/icu4j/main/classes/core/src/com/ibm/icu/util/Calendar.java
+++ b/icu4j/main/classes/core/src/com/ibm/icu/util/Calendar.java
@@ -3076,30 +3076,30 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
         }
 
         // In order to keep the hour invariant (for fields where this is
-        // appropriate), record the DST_OFFSET before and after the add()
-        // operation.  If it has changed, then adjust the millis to
-        // compensate.
-        int dst = 0;
+        // appropriate), check the combined DST & ZONE offset before and
+        // after the add() operation. If it changes, then adjust the millis
+        // to compensate.
+        int prevOffset = 0;
         int hour = 0;
         if (keepHourInvariant) {
-            dst = get(DST_OFFSET) + get(ZONE_OFFSET);
+            prevOffset = get(DST_OFFSET) + get(ZONE_OFFSET);
             hour = internalGet(HOUR_OF_DAY);
         }
 
         setTimeInMillis(getTimeInMillis() + delta);
 
         if (keepHourInvariant) {
-            dst -= get(DST_OFFSET) + get(ZONE_OFFSET);
-            if (dst != 0) {
+            int newOffset = get(DST_OFFSET) + get(ZONE_OFFSET);
+            if (newOffset != prevOffset) {
                 // We have done an hour-invariant adjustment but the
-                // DST offset has altered.  We adjust millis to keep
-                // the hour constant.  In cases such as midnight after
+                // combined offset has changed. We adjust millis to keep
+                // the hour constant. In cases such as midnight after
                 // a DST change which occurs at midnight, there is the
-                // danger of adjusting into a different day.  To avoid
+                // danger of adjusting into a different day. To avoid
                 // this we make the adjustment only if it actually
                 // maintains the hour.
                 long t = time;
-                setTimeInMillis(time + dst);
+                setTimeInMillis(time + prevOffset - newOffset);
                 if (get(HOUR_OF_DAY) != hour) {
                     setTimeInMillis(t);
                 }
-- 
2.40.0