Changeset 33 for trunk/src/helpers/threads.c

Timestamp:

Feb 7, 2001, 7:30:59 PM (25 years ago)

Author:

umoeller

Message:

misc. changes

File:

• trunk/src/helpers/threads.c (modified) (7 diffs)

trunk/src/helpers/threads.c

@@ -182 +182 @@
  *         to the caller.
  *
+ *      This now (V0.9.9) returns the TID of the new thread or
+ *      null on errors.
+ *
  *@@changed V0.9.0 [umoeller]: default stack size raised for Watcom (thanks, Rdiger Ihle)
  *@@changed V0.9.0 [umoeller]: _beginthread is now only called after all variables have been set (thanks, Rdiger Ihle)
@@ -190 +193 @@
  *@@changed V0.9.5 (2000-08-26) [umoeller]: now using PTHREADINFO
  *@@changed V0.9.7 (2000-12-18) [lafaix]: THRF_TRANSIENT support added
- */
-
-BOOL thrCreate(PTHREADINFO pti,     // out: THREADINFO data
-               PTHREADFUNC pfn,     // in: _Optlink thread function
-               PBOOL pfRunning,     // out: variable set to TRUE while thread is running;
-                                    // ptr can be NULL
-               ULONG flFlags,       // in: THRF_* flags
-               ULONG ulData)        // in: user data to be stored in THREADINFO
-{
-    BOOL            rc = FALSE;
+ *@@changed V0.9.9 (2000-02-06) [umoeller]: now returning TID
+ */
+
+ULONG thrCreate(PTHREADINFO pti,     // out: THREADINFO data
+                PTHREADFUNC pfn,     // in: _Optlink thread function
+                PBOOL pfRunning,     // out: variable set to TRUE while thread is running;
+                                     // ptr can be NULL
+                ULONG flFlags,       // in: THRF_* flags
+                ULONG ulData)        // in: user data to be stored in THREADINFO
+{
+    ULONG ulrc = 0;     // V0.9.9 (2000-02-06) [umoeller]
 
     // (2000-12-18) [lafaix] TRANSIENT
@@ -206 +210 @@
         if (pti == NULL)
             pti = (PTHREADINFO) malloc(sizeof(THREADINFO));
-        else
-            return (rc);
     }
 
@@ -223 +225 @@
         pti->ulData = ulData;
 
-        rc = TRUE;
-
         if (flFlags & THRF_WAIT)
             // "Wait" flag set: create an event semaphore which
@@ -233 +233 @@
                                   FALSE)    // not posted (reset)
                     != NO_ERROR)
-                rc = FALSE;
-
-        pti->fExit = FALSE;
-
-        if (rc)
+            {
+                if (flFlags & THRF_TRANSIENT)
+                    free(pti);
+
+                // stop right here
+                pti = NULL;
+            }
+
+        if (pti)
         {
             pti->tid = _beginthread(        // moved, V0.9.0 (hint: Rdiger Ihle)
@@ -244 +248 @@
                                     3*96000, // plenty of stack
                                     pti);   // parameter passed to thread
-            rc = (pti->tid != 0);
-
-            if (rc)
+            ulrc = pti->tid;
+
+            if (ulrc)
                 if (flFlags & THRF_WAIT)
                 {
@@ -257 +261 @@
     }
 
-    return (rc);
+    return (ulrc);
 }