Changeset 63642 in vbox

Timestamp:

Aug 25, 2016 3:43:45 PM (9 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

110369

Message:

microatl.h: InternalRelease should check VBOX_STRICT not DEBUG. Use AssertMsg instead of AssertMsgFailed. Text was also ambiguous.

File:

• trunk/include/VBox/com/microatl.h (modified) (2 diffs)

trunk/include/VBox/com/microatl.h

@@ -27 +27 @@
 #define ___VBox_com_microatl_h
 
+#include <VBox/cdefs.h> /* VBOX_STRICT */
 #include <iprt/assert.h>
 #include <iprt/critsect.h>
@@ -901 +902 @@
     ULONG InternalRelease()
     {
-#ifdef DEBUG
+#ifdef VBOX_STRICT
         LONG c = ThreadModel::Decrement(&m_iRef);
-        if (c < -(LONG_MAX / 2))
-        {
-            AssertMsgFailed(("Release called on object which has been already released\n"));
-        }
+        AssertMsg(c >= -(LONG_MAX / 2), /* See  ~CComObjectNoLock, ~CComObject & ~CComAggObject. */
+                  ("Release called on object which has been already destroyed!\n"));
         return c;
 #else