This is the mail archive of the cygwin-patches 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]

Re: Fix sem_getvalue


On Wed, Sep 25, 2013 at 12:41:17PM +0200, Paul Kunysch wrote:
>> That looks like a reasonable fix.  Did you trace through all of the
>> callers of semaphore::_getvalue to make sure that some of them aren't
>> relying on the old behavior?
>
>I did not look for other callers.
>
>I just wrote a very simple test for sem_getvalue() and copied different 
>cygwin1.dll versions to my test-application.

I was being too vague here.  You made what looks like a reasonable
change but it is to a low-level function which is called by more than
one place.  I was hoping that, since you wanted to see this fixed, you
might also be willing to make sure that your change doesn't break
anything else.  But, I didn't say that so, my bad.

I've checked in a modified version of your patch along with a ChangeLog.

Thanks.

cgf


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