cleanup and changelog entry

BadSingleton · BadSingleton · commit 19150086951e · 2021-01-18T11:08:03.000-05:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -28,6 +28,7 @@ details about the cause of the failure
      to the regular method return value (unless they are passed with `ref` or `out` keyword).
 -   BREAKING: Drop support for the long-deprecated CLR.* prefix.
 -   `PyObject` now implements `IEnumerable<PyObject>` in addition to `IEnumerable`
+-   Replaced the old `__import__` hook hack with a PEP302-style Meta Path Loader
 
 ### Fixed
 
@@ -47,6 +48,7 @@ details about the cause of the failure
 -    Incorrectly using a non-generic type with type parameters now produces a helpful Python error instead of throwing NullReferenceException
 -   `import` may now raise errors with more detail than "No module named X"
 
+
 ### Removed
 
 -   implicit assembly loading (you have to explicitly `clr.AddReference` before doing import)
diff --git a/src/runtime/importhook.cs b/src/runtime/importhook.cs
@@ -8,9 +8,7 @@ namespace Python.Runtime
     /// </summary>
     internal static class ImportHook
     {
-        private static IntPtr py_import;
         private static CLRModule root;
-        private static MethodWrapper hook;
         private static IntPtr py_clr_module;
 
         private static IntPtr module_def = IntPtr.Zero;
@@ -26,20 +24,13 @@ def __init__(self):
 
     @classmethod
     def exec_module(klass, mod):
-        # this method is needed to mark this
-        # loader as a non-legacy loader.
+        # This method needs to exist.
         pass
 
     @classmethod
     def create_module(klass, spec):
         import clr
-        a = clr._LoadClrModule(spec)
-        #mod = getattr(clr, '__imported')
-        print(a)
-        #print(mod)
-        #print(mod is a)
-        #delattr(clr, '__imported')
-        return a
+        return clr._LoadClrModule(spec)
 
 class DotNetFinder(importlib.abc.MetaPathFinder):
     
@@ -50,9 +41,7 @@ def __init__(self):
     @classmethod
     def find_spec(klass, fullname, paths=None, target=None): 
         import clr
-        # print(clr._availableNamespaces)
         if (hasattr(clr, '_availableNamespaces') and fullname in clr._availableNamespaces):
-        #if (clr._NamespaceLoaded(fullname)):
             return importlib.machinery.ModuleSpec(fullname, DotNetLoader(), is_package=True)
         return None
 
diff --git a/src/runtime/moduleobject.cs b/src/runtime/moduleobject.cs
@@ -599,7 +599,11 @@ public static int _AtExit()
         [ForbidPythonThreads]
         public static PyObject _LoadClrModule(PyObject spec)
         {
-            var mod = ImportHook.__import__(spec.GetAttr("name").ToString());
+            ModuleObject mod = null;
+            using (var modname = spec.GetAttr("name"))
+            {
+                mod = ImportHook.__import__(modname.ToString());
+            }
             // We can't return directly a ModuleObject, because the tpHandle is
             // not set, but we can return a PyObject.
             return new PyObject(mod.pyHandle);
