This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

sem_getvalue patch revisited


Sam Robb posted a semaphore getvalue patch on June 7th, 2002:

http://cygwin.com/ml/cygwin-patches/2002-q2/msg00339.html

I had been working on the same problem, but I produced a different
solution.  Please see the attachment, which was written against 1.3.10-1
but also applies to 1.3.11-3 cleanly.

In particular, Rob's patch returns `(*sem)->currentvalue` directly from
inside the new __sem_ function, whereas my patch adds a GetValue method to
the semaphore class.  (I was trying to follow the style of the existing
Cygwin semaphore.)

I am wondering whether Sam's patch is functionally equivalent to my own,
and whether one is more or less correct than the other.

[]

Attachment: cygwin-sem_getvalue-dajhorn.patch
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]