[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: r952 - in trunk: sptk4 src/utils
- To: SPTK discussion <sptk@total-knowledge.com>
- Subject: Re: r952 - in trunk: sptk4 src/utils
- From: Alexey Parshin <alexeyp@gmail.com>
- Date: Wed, 30 Sep 2009 09:47:08 +0000
- Delivered-to: mailing list sptk@total-knowledge.com
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type; bh=ZdibrYTFLMicI3epFZxyjeVJV4xFsur5ySb1XzqbtNk=; b=omFA9ojheJj0EbKvf/vwicC7zdj4XR3lrSSGJnWc3dBjKwXKkgDpvtstk0w6Wsd/f+ yRI2l3T+Y2yFsfFRbgtJXmRZUeDnuFR1rKmXv0asdsFCdHh+ldqj8VvflMr9ZvtTP7Wh +zybcCgV2mRLSa2D/BXdV17DxYOCiKDv38WTA=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; b=U60t6ZWsAU999lHEO5dzPT+5BXEH4m3Uod5aocT8YH9wU4tsI21eMr19QKrY6eustn zXdIYvcNv3SlJrOYTUOHlzj+FUsuDAe7BCs1HM+2Be5xfkDwQzbYatUmO2LyIjeHQooU L6NdW0p9jgs/Is4CHO3mAxe9BcOemhegCNJKM=
- In-reply-to: <4AC2DABD.9010202@total-knowledge.com>
- Mailing-list: contact sptk-help@total-knowledge.com; run by ezmlm
- References: <courier.000000004AC28816.00000E27@mail.total-knowledge.com> <4AC2DABD.9010202@total-knowledge.com>
I'm trying to make the CWaiter class more correct. So, the form of the
sendSignal that doesn't lock is incorrect. It isn't used internally in SPTK,
or any of my projects.I can return it back, if you need it, though.
2009/9/30 Ilya A. Volynets-Evenbakh <ilya@total-knowledge.com>
> alexey@mail.total-knowledge.com wrote:
> > Removed dead method CWaiter::sendSignalNoLock().
> >
> It's not a dead method. It's the sendSignal, which is a useless one from
> mutex semantics point of view.
>
> Generally correct way to deal with signal is
> - lock mutex
> - do something
> - send signal
> - unlock mutex
>
> Otherwise you get race conditions.
>
> --
> Ilya A. Volynets-Evenbakh
> http://www.total-knowledge.com
>
>
--
Alexey Parshin,
http://www.sptk.net
List hosted by Total Knowledge
Authoright © Total Knowledge: 2005