Merge pull request #934 from kmod/set_fix

Try to fix set-dtor segfault

src/runtime/set.cpp

@@ -664,7 +664,8 @@ done:
 } // namespace set
 
 void BoxedSet::dealloc(Box* b) noexcept {
-    assert(PyAnySet_Check(b));
+    // Unfortunately, this assert requires accessing the type object, which might have been freed already:
+    // assert(PyAnySet_Check(b));
     static_cast<BoxedSet*>(b)->s.freeAllMemory();
 }
 

src/runtime/types.h

@@ -209,7 +209,7 @@ public:
 
     // A "safe" tp_dealloc destructor/finalizer is one we believe:
     //  1) Can be called at any point after the object is dead.
-    //      (implies it's references could be finalized already)
+    //      (implies it's references could be finalized already, including its class)
     //  2) Won't take a lot of time to run.
     //  3) Won't take up a lot of memory (requiring another GC run).
     //  4) Won't resurrect itself.