Changeset 20970 in webkit for trunk/JavaScriptCore

Timestamp:

Apr 20, 2007, 12:50:14 AM (18 years ago)

Author:

bdash

Message:

2007-04-20 Mark Rowe <​[email protected]>

Reviewed by Anders.

Improve FreeBSD compatibility, as suggested by Alexander Botero-Lowry.

• kjs/collector.cpp: (KJS::currentThreadStackBase): FreeBSD requires that pthread_attr_t's are initialized via pthread_attr_init before being used in any context.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/collector.cpp (modified) (2 diffs)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2007-04-20  Mark Rowe  <[email protected]>
+
+        Reviewed by Anders.
+
+        Improve FreeBSD compatibility, as suggested by Alexander Botero-Lowry.
+
+        * kjs/collector.cpp:
+        (KJS::currentThreadStackBase): FreeBSD requires that pthread_attr_t's are
+        initialized via pthread_attr_init before being used in any context.
+
 2007-04-19  Mark Rowe  <[email protected]>
 

trunk/JavaScriptCore/kjs/collector.cpp

@@ -241 +241 @@
     if (stackBase == 0 || thread != stackThread) {
         pthread_attr_t sattr;
+        pthread_attr_init(&sattr);
 #if HAVE(PTHREAD_NP_H)
         // e.g. on FreeBSD 5.4, [email protected]
@@ -252 +253 @@
         (void)rc; // FIXME: deal with error code somehow?  seems fatal...
         ASSERT(stackBase);
+        pthread_attr_destroy(&sattr);
         return (void*)(size_t(stackBase) + stackSize);
         stackThread = thread;