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]

Re: [PATCH] fix segv in pthread_mutex::init



On Thu, 17 Oct 2002, Robert Collins wrote:

> On Thu, 2002-10-17 at 18:11, Thomas Pfaff wrote:
> >
> > This patch should fix the segfault in pthread_mutex::init by changing the
> > test order for a valid object and checking for valid initializer object
> > first..
>
> I'm happy with the verifyable_object change. I'm not happy with the
> pthread_mutex::init change (yet).
>
> I've checked in the verifyable_object stuff, along with a
> pthread_mutex::init change I am happy with.
>
> The reason I wasn't happy with your change was threefold:
> 1) there is no need to add scoping to static calls within a class. Doing
> so reduces readability, and should be avoided.

I am sorry, but i have first created the patch in my 1.3.12-4 version and
than ported to the CVS version. I have corrected it in my second posting.

I am still using 1.3.12-4 until i have ported my MTinterface and mutex
patches to the latest version.

> 2) The test was no longer *obvious* at first read, whereas (IMO) what
> I've checked in is.
>

The isGoodInitializerOrBadObject method looks good.

Thomas


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