added debug message to the assert that fails on Mac to figure out the cause

lostmsu · lostmsu · commit da835c9f3bee · 2021-09-22T15:12:14.000-07:00
diff --git a/src/runtime/classbase.cs b/src/runtime/classbase.cs
@@ -387,13 +387,13 @@ static int ClearImpl(IntPtr ob, ManagedType self)
             bool isTypeObject = Runtime.PyObject_TYPE(ob) == Runtime.PyCLRMetaType;
             if (!isTypeObject)
             {
+                ClearObjectDict(ob);
+
                 int baseClearResult = BaseUnmanagedClear(ob);
                 if (baseClearResult != 0)
                 {
                     return baseClearResult;
                 }
-
-                ClearObjectDict(ob);
             }
             if (self is not null) self.tpHandle = IntPtr.Zero;
             return 0;
diff --git a/src/runtime/classderived.cs b/src/runtime/classderived.cs
@@ -76,7 +76,15 @@ internal ClassDerivedObject(Type tp) : base(tp)
             // So we don't call PyObject_GC_Del here and instead we set the python
             // reference to a weak reference so that the C# object can be collected.
             GCHandle gc = GCHandle.Alloc(self, GCHandleType.Weak);
-            Debug.Assert(self.TypeReference == Runtime.PyObject_TYPE(self.ObjectReference));
+#if DEBUG
+            BorrowedReference instanceType = Runtime.PyObject_TYPE(self.ObjectReference);
+            if (self.TypeReference != instanceType)
+            {
+                using var type = new PyObject(self.TypeReference);
+                using var instType = new PyObject(instanceType);
+                Debug.Fail($"{type} vs {instType}");
+            }
+#endif
             SetGCHandle(self.ObjectReference, self.TypeReference, gc);
             self.gcHandle.Free();
             self.gcHandle = gc;

Original file line number	Diff line number	Diff line change
`@@ -387,13 +387,13 @@ static int ClearImpl(IntPtr ob, ManagedType self)`
`387`	`387`	`bool isTypeObject = Runtime.PyObject_TYPE(ob) == Runtime.PyCLRMetaType;`
`388`	`388`	`if (!isTypeObject)`
`389`	`389`	`{`
	`390`	`+ ClearObjectDict(ob);`
	`391`	`+`
`390`	`392`	`int baseClearResult = BaseUnmanagedClear(ob);`
`391`	`393`	`if (baseClearResult != 0)`
`392`	`394`	`{`
`393`	`395`	`return baseClearResult;`
`394`	`396`	`}`
`395`		`-`
`396`		`- ClearObjectDict(ob);`
`397`	`397`	`}`
`398`	`398`	`if (self is not null) self.tpHandle = IntPtr.Zero;`
`399`	`399`	`return 0;`