[PATCH] fix segv in pthread_mutex::init
Robert Collins
rbcollins@cygwin.com
Thu Oct 17 01:46:00 GMT 2002
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.
2) The test was no longer *obvious* at first read, whereas (IMO) what
I've checked in is.
Cheers,
Rob
--
---
GPG key available at: http://users.bigpond.net.au/robertc/keys.txt.
---
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20021017/13f24d05/attachment.sig>
More information about the Cygwin-patches
mailing list