Reviewed by Adam.
authorkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Oct 2006 01:42:00 +0000 (01:42 +0000)
committerkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Oct 2006 01:42:00 +0000 (01:42 +0000)
        - Removed an unnecessary assert that was stopping many pages.  tm_gmtoff was not set for UTC time in mozilla but is always set for us.

        * kjs/DateMath.cpp:
        (KJS::getUTCOffset):
        (KJS::msToTM):
        * kjs/date_object.cpp:
        (KJS::gmtoffset):
        (KJS::formatTime):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@16798 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/DateMath.cpp
JavaScriptCore/kjs/date_object.cpp

index 7383113ea9288fdff94e11e28a844eebc1ab475d..9f2f8d82c0671a2cfc5902d75aa06e96e9ffd18f 100644 (file)
@@ -1,3 +1,16 @@
+2006-10-04  Kevin McCullough  <KMcCullough@apple.com>
+
+        Reviewed by Adam.
+
+        - Removed an unnecessary assert that was stopping many pages.  tm_gmtoff was not set for UTC time in mozilla but is always set for us.
+
+        * kjs/DateMath.cpp:
+        (KJS::getUTCOffset):
+        (KJS::msToTM):
+        * kjs/date_object.cpp:
+        (KJS::gmtoffset):
+        (KJS::formatTime):
+
 2006-10-04  Geoffrey Garen  <ggaren@apple.com>
 
         Patch by Darin and me, reviewed by Maciej.
index d6724d7e2f16f58e5c162e07c9370dcdd05c4cd6..193bd678ac1acb2cf937f26b895024bc033386bf 100644 (file)
@@ -333,10 +333,10 @@ double getUTCOffset() {
         ltime.tm_mday = 2;
         ltime.tm_year = 70;
 
-        #if !PLATFORM(WIN_OS)
+#if !PLATFORM(WIN_OS)
         ltime.tm_zone = 0;
         ltime.tm_gmtoff = 0;
-        #endif
+#endif
 
         utcOffset = mktime(&ltime) - (hoursPerDay * secondsPerHour);
         utcOffset *= -msPerSecond;
@@ -445,14 +445,14 @@ void msToTM(double ms, bool outputIsUTC, struct tm& tm)
     tm.tm_isdst =  isDST(ms);
 
     //everyone else seems to have these fields
-    #if !PLATFORM(WIN_OS)
+#if !PLATFORM(WIN_OS)
     struct tm xtm;
     // FIXME: time_t has a potential problem in 2038
     time_t seconds = static_cast<time_t>(ms/usecPerMsec);
     localtime_r(&seconds, &xtm);
     tm.tm_gmtoff = xtm.tm_gmtoff;
     tm.tm_zone = xtm.tm_zone;
-    #endif
+#endif
 }
 
 bool isDST(const double& ms)
index 1539173cad0bb9a7d0060f959d65fe690b4a9bbd..2c8be277071e1e75037124031669cc616bc129ad 100644 (file)
@@ -69,7 +69,8 @@ inline int gmtoffset(const tm& t)
 {
 #if PLATFORM(WIN_OS)
     // Time is supposed to be in the current timezone.
-    return static_cast<int>(getUTCOffset()/1000.0);
+    static int utcOffset = static_cast<int>(getUTCOffset()/1000.0);
+    return utcOffset;
 #else
     return t.tm_gmtoff;
 #endif
@@ -183,9 +184,6 @@ static UString formatTime(const tm &t, bool utc)
 {
     char buffer[100];
     if (utc) {
-#if !PLATFORM(WIN_OS)   //win doesn't have the tm_gtoff member
-        ASSERT(t.tm_gmtoff == 0);
-#endif
         snprintf(buffer, sizeof(buffer), "%02d:%02d:%02d GMT", t.tm_hour, t.tm_min, t.tm_sec);
     } else {
         int offset = abs(gmtoffset(t));