Changeset 29454 in webkit for trunk/JavaScriptCore/kjs/DateMath.h

Timestamp:

Jan 13, 2008, 11:47:56 AM (17 years ago)

Author:

Darin Adler

Message:

Contributions and review by Adriaan de Groot,
Simon Hausmann, Eric Seidel, and Darin Adler.

• ​http://bugs.webkit.org/show_bug.cgi?id=16590 Compilation fixes for Solaris.

• kjs/DateMath.h: (KJS::GregorianDateTime::GregorianDateTime): Use the WIN_OS code path for SOLARIS too, presumably because Solaris also lacks the tm_gtoff and tm_zone fields. (KJS::GregorianDateTime::operator tm): Ditto.

• kjs/collector.cpp: (KJS::currentThreadStackBase): Use thr_stksegment on Solaris.

• wtf/MathExtras.h: (isfinite): Implement for Solaris. (isinf): Ditto. (signbit): Ditto. But this one is wrong, so I added a FIXME.

• wtf/Platform.h: Define PLATFORM(SOLARIS) when "sun" or "sun" is defined.

File:

• trunk/JavaScriptCore/kjs/DateMath.h (modified) (3 diffs)

trunk/JavaScriptCore/kjs/DateMath.h

@@ -69 +69 @@
 
 // Intentionally overridding the default tm of the system
-// Not all OS' have the same members of their tm's
+// Tee members of tm differ on various operating systems.
 struct GregorianDateTime : Noncopyable {
     GregorianDateTime()
@@ -102 +102 @@
         , isDST(inTm.tm_isdst)
     {
-#if !PLATFORM(WIN_OS)
+#if !PLATFORM(WIN_OS) && !PLATFORM(SOLARIS)
         utcOffset = static_cast<int>(inTm.tm_gmtoff);
 
@@ -129 +129 @@
         ret.tm_isdst =  isDST;
 
-#if !PLATFORM(WIN_OS)
+#if !PLATFORM(WIN_OS) && !PLATFORM(SOLARIS)
         ret.tm_gmtoff = static_cast<long>(utcOffset);
         ret.tm_zone = timeZone;