]> granicus.if.org Git - graphviz/commitdiff
remove SFMTXUNLOCK() no-op
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 17 Oct 2020 20:01:51 +0000 (13:01 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 24 Oct 2020 02:16:10 +0000 (19:16 -0700)
lib/sfio/sfhdr.h
lib/sfio/sfmode.c
lib/sfio/sfstack.c

index 7db0b8d577bf193a38144bd1398e1854bb013b52..83bcca2c4e1169b96379ebcd2a10dc853d8fc658 100644 (file)
@@ -85,7 +85,6 @@ extern "C" {
 #undef SF_MTSAFE               /* no need to worry about thread-safety */
 #define SF_MTSAFE              0
 
-#define SFMTXUNLOCK(f)         (void)(0)
 #define SFMTXSTART(f,v)                { if(!f) return(v); }
 #define SFMTXRETURN(f,v)       { return(v); }
 
index 657855c2c7c0e6ab68413b02c958b08f1a5bb27b..479d2c81a4f83f7c6cf6223b419457bdc417a80e 100644 (file)
@@ -77,7 +77,6 @@ static void _sfcleanup(void)
                (void) SFSETBUF(f, NIL(void *), 0);
            f->mode |= pool;
 
-           SFMTXUNLOCK(f);
            SFOPEN(f, 0);
        }
     }
index 2c765c4966da900d0c35c023ac3e7e1b5879ff2e..c5f9462c64db6414030117d49698ee8ef3e501df 100644 (file)
  *************************************************************************/
 
 #include       <sfio/sfhdr.h>
-
+#include <stddef.h>
 
 /*     Push/pop streams
 **
 **     Written by Kiem-Phong Vo.
 */
 
-#define STKMTXRETURN(f1,f2,rv) \
-       { if(f1) SFMTXUNLOCK(f1); \
-         if(f2) SFMTXUNLOCK(f2); \
-         return(rv); \
-       }
-
 /**
  * @param f1 base of stack
  * @param f2 top of stack
@@ -36,22 +30,22 @@ Sfio_t *sfstack(Sfio_t * f1, Sfio_t * f2)
     Sfrsrv_t *rsrv;
 
     if (f1 && (f1->mode & SF_RDWR) != f1->mode && _sfmode(f1, 0, 0) < 0)
-       STKMTXRETURN(f1, f2, NIL(Sfio_t *));
+       return NULL;
     if (f2 && (f2->mode & SF_RDWR) != f2->mode && _sfmode(f2, 0, 0) < 0)
-       STKMTXRETURN(f1, f2, NIL(Sfio_t *));
+       return NULL;
     if (!f1)
-       STKMTXRETURN(f1, f2, f2);
+       return f2;
 
     /* give access to other internal functions */
     _Sfstack = sfstack;
 
     if (f2 == SF_POPSTACK) {
        if (!(f2 = f1->push))
-           STKMTXRETURN(f1, f2, NIL(Sfio_t *));
+           return NULL;
        f2->mode &= ~SF_PUSH;
     } else {
        if (f2->push)
-           STKMTXRETURN(f1, f2, NIL(Sfio_t *));
+           return NULL;
        if (f1->pool && f1->pool != &_Sfpool && f1->pool != f2->pool && f1 == f1->pool->sf[0]) {        /* get something else to pool front since f1 will be locked */
            for (n = 1; n < f1->pool->n_sf; ++n) {
                if (SFFROZEN(f1->pool->sf[n]))
@@ -89,5 +83,5 @@ Sfio_t *sfstack(Sfio_t * f1, Sfio_t * f2)
     SFOPEN(f1, 0);
     SFOPEN(f2, 0);
 
-    STKMTXRETURN(f1, f2, rf);
+    return rf;
 }