Update comment of ClassManager.RemoveClasses

amos402 · amos402 · commit fd2b66228dd1 · 2020-08-27T10:27:03.000+08:00
diff --git a/src/runtime/classmanager.cs b/src/runtime/classmanager.cs
@@ -53,8 +53,11 @@ internal static void RemoveClasses()
             {
                 foreach (var cls in cache.Values)
                 {
+                    // XXX: Force to release instance's managed resources
+                    // but not dealloc itself immediately.
+                    // These managed resources should preserve vacant shells
+                    // since others may still referencing it.
                     cls.CallTypeTraverse(OnVisit, visitedPtr);
-                    // XXX: Force release instance resources but not dealloc itself.
                     cls.CallTypeClear();
                     cls.DecrRefCount();
                 }

Original file line number	Diff line number	Diff line change
`@@ -53,8 +53,11 @@ internal static void RemoveClasses()`
`53`	`53`	`{`
`54`	`54`	`foreach (var cls in cache.Values)`
`55`	`55`	`{`
	`56`	`+ // XXX: Force to release instance's managed resources`
	`57`	`+ // but not dealloc itself immediately.`
	`58`	`+ // These managed resources should preserve vacant shells`
	`59`	`+ // since others may still referencing it.`
`56`	`60`	`cls.CallTypeTraverse(OnVisit, visitedPtr);`
`57`		`- // XXX: Force release instance resources but not dealloc itself.`
`58`	`61`	`cls.CallTypeClear();`
`59`	`62`	`cls.DecrRefCount();`
`60`	`63`	`}`