[Python-checkins] r52566 - sandbox/trunk/import_in_py/importer.py

[brett.cannon]

Author: brett.cannon
Date: Tue Oct 31 23:46:25 2006
New Revision: 52566

Modified:
   sandbox/trunk/import_in_py/importer.py
Log:
Tweak some comments.


Modified: sandbox/trunk/import_in_py/importer.py
==============================================================================
--- sandbox/trunk/import_in_py/importer.py	(original)
+++ sandbox/trunk/import_in_py/importer.py	Tue Oct 31 23:46:25 2006
@@ -43,7 +43,7 @@
   bound in the namespace (including previously imported modules)
   [package essay].
 * Must make sure that importing submodules later on have the proper attributes
-  for modules with the submodule in its path (e.g., inheriting A.B.C and A.B.D
+  for modules with the submodule in its path (e.g., importing A.B.C and A.B.D
   need to have A.B having C and D as attributes) [package essay].
 * Submodules must be imported first (e.g., importing A.B requires A to be
   imported first) [package essay].
@@ -69,7 +69,7 @@
 Improvements
 ------------
 * Have a meta_path entry for checking sys.modules to remove need for
-  loaders to do it.
+  loaders or __import__ to do it.
 * Put importer objects directly into sys.path to remove need for
   sys.path_importer_cache.  Could leave string entries on sys.path that do
   not have an importer so they can be re-checked the next time a new import
@@ -86,9 +86,9 @@
       no real need.
     + Importing the module being imported in a circular import dependency
       requires that module added to sys.modules stay consistent from the
-      point it is added to initialization.
-* Remove any idea of default importer.
-    + Removes None entries from sys.path_importer_cache.
+      point it is added to initialization anyway.
+* Remove any idea of a default importer.
+    + Removes support for None entries from sys.path_importer_cache.
     + Rely on default importers being in sys.path_hooks or sys.meta_path. 
        
 Rejected Ideas
@@ -196,7 +196,7 @@
 
     """sys.meta_path class for importing frozen modules."""
     
-    # XXX Frozen packages seem to exist.
+    # XXX Need to make sure frozen packages can be handled.
 
     _find = imp.is_frozen
     _load = imp.init_frozen