Rename ClassManager.RemoveClasses

BadSingleton · BadSingleton · commit bfbf2c3e6d35 · 2020-08-20T15:06:52.000-04:00
To `DisposePythonWrappersForClrTypes` Addresses comment: #958
diff --git a/src/runtime/classmanager.cs b/src/runtime/classmanager.cs
@@ -39,7 +39,7 @@ public static void Reset()
             cache = new Dictionary<Type, ClassBase>(128);
         }
 
-        internal static void RemoveClasses()
+        internal static void DisposePythonWrappersForClrTypes()
         {
             var visited = new HashSet<IntPtr>();
             var visitedHandle = GCHandle.Alloc(visited);
diff --git a/src/runtime/runtime.cs b/src/runtime/runtime.cs
@@ -334,7 +334,7 @@ internal static void Shutdown()
             RemoveClrRootModule();
 
             MoveClrInstancesOnwershipToPython();
-            ClassManager.RemoveClasses();
+            ClassManager.DisposePythonWrappersForClrTypes();
             TypeManager.RemoveTypes();
 
             MetaType.Release();

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public static void Reset()`
`39`	`39`	`cache = new Dictionary<Type, ClassBase>(128);`
`40`	`40`	`}`
`41`	`41`
`42`		`- internal static void RemoveClasses()`
	`42`	`+ internal static void DisposePythonWrappersForClrTypes()`
`43`	`43`	`{`
`44`	`44`	`var visited = new HashSet<IntPtr>();`
`45`	`45`	`var visitedHandle = GCHandle.Alloc(visited);`