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

  • From: Colin Günther <coling@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 02 Dec 2009 14:05:45 +0100

Ingo Weinhold schrieb:
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().
I will look into it, but I must commit I don't quite see how the memory de-/allocation can be ommited with allowing C++ structure members in c only structures. So it will take some time for me understand this one.
CU, Ingo


-Colin

Other related posts: