Changeset 78 for trunk/src/helpers/semaphores.c

Timestamp:

May 28, 2001, 9:47:12 AM (24 years ago)

Author:

umoeller

Message:

Misc changes.

File:

• trunk/src/helpers/semaphores.c (modified) (4 diffs)

trunk/src/helpers/semaphores.c

@@ -169 +169 @@
  *      I have created, this returns an APIRET.
  *
- *@@added V0.9.12 (2001-05-27) [umoeller]
  */
 
@@ -187 +186 @@
  *@@ UnlockGlobal:
  *
- *@@added V0.9.12 (2001-05-27) [umoeller]
  */
 
@@ -258 +256 @@
  *          was set to NULLHANDLE.
  *
- *      --  ERROR_SEM_BUSY: couldn't delete one
- *          of the member semaphores because they
- *          were held by some other thread.
- */
-
-APIRET semDeleteRWMutex(PHRW phrw)
+ *      --  ERROR_SEM_BUSY: semaphore is currently
+ *          requested.
+ */
+
+APIRET semDeleteRWMutex(PHRW phrw)      // in/out: rwsem handle
 {
     APIRET  arc = NO_ERROR;
@@ -274 +271 @@
            )
         {
-            if (    (!(arc = DosCloseEventSem(pMutex->hevWriterDone)))
-                 && (!(arc = DosCloseEventSem(pMutex->hevReadersDone)))
+            if (    (pMutex->cReaders)
+                 || (pMutex->cWriters)
                )
+                arc = ERROR_SEM_BUSY;
+            else
             {
-                ULONG cItems = pMutex->cReaderThreads;
-                TREE **papNodes = treeBuildArray(pMutex->ReaderThreadsTree,
-                                                 &cItems);
-                if (papNodes)
+                if (    (!(arc = DosCloseEventSem(pMutex->hevWriterDone)))
+                     && (!(arc = DosCloseEventSem(pMutex->hevReadersDone)))
+                   )
                 {
-                    ULONG ul;
-                    for (ul = 0; ul < cItems; ul++)
-                        free(papNodes[ul]);
-
-                    free(papNodes);
+                    ULONG cItems = pMutex->cReaderThreads;
+                    TREE **papNodes = treeBuildArray(pMutex->ReaderThreadsTree,
+                                                     &cItems);
+                    if (papNodes)
+                    {
+                        ULONG ul;
+                        for (ul = 0; ul < cItems; ul++)
+                            free(papNodes[ul]);
+
+                        free(papNodes);
+                    }
+
+                    free(pMutex);
+                    *phrw = NULLHANDLE;
                 }
-
-                free(pMutex);
-                *phrw = NULLHANDLE;
             }
         }