Changeset 17444 in webkit for trunk/JavaScriptCore/kjs/date_object.cpp

Timestamp:

Oct 30, 2006, 1:34:54 PM (19 years ago)

Author:

kmccullo

Message:

Reviewed by Adam.

• Fix some timezone issues and JavaScriptCore date tests. Addresses bugzilla 4930.

• kjs/DateMath.h: (KJS::GregorianDateTime::GregorianDateTime): Here's the fix, to add parenthesis for order of precedence.
• kjs/date_object.cpp: (KJS::DateProtoFunc::callAsFunction): (KJS::DateObjectImp::construct): memset not needed as GregorianDateTime initializes itself.

File:

• trunk/JavaScriptCore/kjs/date_object.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/date_object.cpp

@@ -552 +552 @@
     return jsNumber(ms);
   case GetTimezoneOffset:
-    return jsNumber(-gmtoffset(t) / 60);
+    return jsNumber(-gmtoffset(t) / minutesPerHour);
   case SetTime:
     milli = roundValue(exec, args[0]);
@@ -664 +664 @@
     } else {
       GregorianDateTime t;
-      memset(&t, 0, sizeof(t));
       int year = args[0]->toInt32(exec);
       t.year = (year >= 0 && year <= 99) ? year : year - 1900;