Changeset 31057 in webkit for trunk/JavaScriptCore/kjs/DateMath.cpp

Timestamp:

Mar 14, 2008, 7:59:38 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin.

Get rid of a localime() call on platforms that have better alternatives.

• kjs/DateMath.h: Added getLocalTime();

• kjs/DateMath.cpp: (KJS::getLocalTime): (KJS::getDSTOffsetSimple): Implementation moved from getDSTOffsetSimple().

• kjs/date_object.cpp: (KJS::DateObjectImp::callAsFunction): Switched to getLocalTime().

File:

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

trunk/JavaScriptCore/kjs/DateMath.cpp

@@ -293 +293 @@
 }
 
+void getLocalTime(const time_t* localTime, struct tm* localTM)
+{
+#if PLATFORM(QT)
+#if USE(MULTIPLE_THREADS)
+#error Mulitple threads are currently not supported in the Qt/mingw build
+#endif
+    *localTM = *localtime(localTime);
+#elif PLATFORM(WIN_OS)
+    #if COMPILER(MSVC7)
+    *localTM = *localtime(localTime);
+    #else
+    localtime_s(localTM, localTime);
+    #endif
+#else
+    localtime_r(localTime, localTM);
+#endif
+}
+
 // There is a hard limit at 2038 that we currently do not have a workaround
 // for (rdar://problem/5052975).
@@ -416 +434 @@
 
     tm localTM;
-#if PLATFORM(QT)
-    // ### this is not threadsafe but we don't use multiple threads anyway
-    // in the Qt build
-#if USE(MULTIPLE_THREADS)
-#error Mulitple threads are currently not supported in the Qt/mingw build
-#endif
-    localTM = *localtime(&localTime);
-#elif PLATFORM(WIN_OS)
-    #if COMPILER(MSVC7)
-    localTM = *localtime(&localTime);
-    #else
-    localtime_s(&localTM, &localTime);
-    #endif
-#else
-    localtime_r(&localTime, &localTM);
-#endif
+    getLocalTime(&localTime, &localTM);
 
     double diff = ((localTM.tm_hour - offsetHour) * secondsPerHour) + ((localTM.tm_min - offsetMinute) * 60);