[haiku-commits] Re: r34427 - in haiku/trunk/src/libs/compat/freebsd_network: . compat/sys

  • From: Ingo Weinhold <ingo_weinhold@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 02 Dec 2009 11:59:11 +0100

On 2009-12-02 at 09:03:56 [+0100], Ingo Weinhold <ingo_weinhold@xxxxxx> 
wrote:
> 
> On 2009-12-01 at 19:48:54 [+0100], coling@xxxxxx wrote:
> > Author: colin
> > Date: 2009-12-01 19:48:54 +0100 (Tue, 01 Dec 2009)
> > New Revision: 34427
> > Changeset: http://dev.haiku-os.org/changeset/34427/haiku
> > 
> > Modified:
> >    haiku/trunk/src/libs/compat/freebsd_network/Condvar.cpp
> >    haiku/trunk/src/libs/compat/freebsd_network/compat/sys/condvar.h
> >    haiku/trunk/src/libs/compat/freebsd_network/condvar.c
> > Log:
> > * Getting rid of the necessity for cleaning up initialized
> > ConditionalVariables
> >   in uninit_condition_variables(). This method and its counterpart
> >   init_condition_variables() will be removed, once the need for the
> >   sConditionVariableHash will be eliminated.
> > * Now every initialized  ConditionalVariable gets removed right after it 
> > is
> > no
> >   longer needed. Ingo what do you think?
> 
> Yep, looks better. I'll look into implementing some trickery to allow C++
> structure members in C only structures. Then you can also get rid of the
> memory allocation/deletion in _cv_init()/_cv_destroy().

Implemented in r34441, but it might be unhandy to use. It's up to you 
whether you want to use it or not.

CU, Ingo

Other related posts: