[prev in list] [next in list] [prev in thread] [next in thread] 

List:       vbox-dev
Subject:    Re: [vbox-dev] A harddisk seems to be forced to have a parent,
From:       Mark Cranness <mark.cranness () gmail ! com>
Date:       2010-02-12 5:13:05
Message-ID: 2d83f4251002112113sa809f2cse6289c4d567c5ac2 () mail ! gmail ! com
[Download RAW message or body]

On 11 February 2010 23:17, Christophe Devriese
<christophe.devriese@gmail.com> wrote:
> Seems to me that it does.

> On Thu, Feb 11, 2010 at 8:48 AM, Achim Hasenmueller <achim@sun.com> wrote:
>>
>> The assertion does not trigger if mParent == NULL, right?
>> Achim.

Try replacing it with:

    Assert ( (!parent.isNull() || mParent == NULL) );

If the original (without the double brackets) triggers the Assert when
mParent == NULL, but the version with double-brackets does not, then
you have a bad definition of the Assert macro.

I note that COMDefs.h includes this:

+++++++++++++++
/* Both VBox/com/assert.h and qglobal.h contain a definition of ASSERT.
 * Either of them can be already included here, so try to shut them up.  */
#undef ASSERT

#include <VBox/com/com.h>
#include <VBox/com/array.h>
#include <VBox/com/assert.h>

#undef ASSERT
+++++++++++++++

@Devs: Wouldn't the second '#undef ASSERT' be a problem...

Mark

_______________________________________________
vbox-dev mailing list
vbox-dev@virtualbox.org
http://vbox.innotek.de/mailman/listinfo/vbox-dev
[prev in list] [next in list] [prev in thread] [next in thread]