Re: libpq thread-locking - Mailing list pgsql-patches

From Andrew Chernow
Subject Re: libpq thread-locking
Date
Msg-id 482D9996.5000406@esilo.com
Whole thread Raw
In response to Re: libpq thread-locking  (Magnus Hagander <magnus@hagander.net>)
Responses Re: libpq thread-locking
List pgsql-patches
! int
   pthread_mutex_init(pthread_mutex_t *mp, void *attr)
   {
       *mp = CreateMutex(0, 0, 0);
+     if (*mp == NULL)
+         return 1;
+     return 0;
   }

Maybe it would be better to emulate what pthreads does.  Instead of
returing 1 to indicate an error, return an errno.  In the above case,
ENOMEM seems like a good fit.

Also, maybe you should check the passed in mutex pointer.  If its NULL,
you could return EINVAL.

--
Andrew Chernow
eSilo, LLC
every bit counts
http://www.esilo.com/

pgsql-patches by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: libpq thread-locking
Next
From: Magnus Hagander
Date:
Subject: Re: libpq thread-locking