- Fix tab-completion so that quotes are not closed unless the completion is

unambiguous.

- Small cleanups in ipy_stock_completers.

IPython/Extensions/ipy_stock_completers.py

@@ -49,16 +49,16 b' pkg_cache = None'
 
 def module_completer(self,event):    
     """ Give completions after user has typed 'import' """
-    
+    
     # only a local version for py 2.4, pkgutil has no walk_packages() there
-    if sys.version_info < (2,5):
-        for el in [f[:-3] for f in glob.glob("*.py")]:
-            yield el
+    if sys.version_info < (2,5):
+        for el in [f[:-3] for f in glob.glob("*.py")]:
+            yield el
         return
-
+
     global pkg_cache
     import pkgutil,imp,time
-    #current = 
+    #current = 
     if pkg_cache is None:
         print "\n\n[Standby while scanning modules, this can take a while]\n\n"
         pkg_cache = list(pkgutil.walk_packages())
@@ -95,11 +95,14 b' def runlistpy(self, event):'
     comps = shlex.split(event.line)
     relpath = (len(comps) > 1 and comps[-1] or '')
    
-    print "rp",relpath
+    #print "rp",relpath  # dbg
+    glob = glob.glob
+    isdir = os.path.isdir
     if relpath.startswith('~'):
         relpath = os.path.expanduser(relpath)
-    dirs = [f.replace('\\','/') + "/" for f in  glob.glob(relpath+'*') if os.path.isdir(f)]
-    pys =  [f.replace('\\','/') for f in  glob.glob(relpath+'*.py')]
+    dirs = [f.replace('\\','/') + "/" for f in glob(relpath+'*')
+            if isdir(f)]
+    pys =  [f.replace('\\','/') for f in glob(relpath+'*.py')]
     return dirs + pys
 
 ip.set_hook('complete_command', runlistpy, str_key = '%run')
@@ -127,4 +130,4 b' def cd_completer(self, event):'
         return [relpath]
     return found
 
-ip.set_hook('complete_command', cd_completer, str_key = '%cd')
+ip.set_hook('complete_command', cd_completer, str_key = '%cd')

IPython/completer.py

@@ -344,7 +344,7 b' class IPCompleter(Completer):'
         current (as of Python 2.3) Python readline it's possible to do
         better."""
 
-        # print 'Completer->file_matches: <%s>' % text # dbg
+        #print 'Completer->file_matches: <%s>' % text # dbg
 
         # chars that require escaping with backslash - i.e. chars
         # that readline treats incorrectly as delimiters, but we
@@ -406,7 +406,9 b' class IPCompleter(Completer):'
             # Takes care of links to directories also.  Use '/'
             # explicitly, even under Windows, so that name completions
             # don't end up escaped.
-            matches[0] += '/'
+            pjoin = os.path.join
+            d = matches[0]
+            matches = [ pjoin(d,p) for p in os.listdir(d) ]
         return matches
 
     def alias_matches(self, text):

doc/ChangeLog

@@ -1,3 +1,10 b''
[phone]  Fernando Perez  <[e-mail]>
+
+	* IPython/completer.py (IPCompleter.file_matches): Fix
+	tab-completion so that quotes are not closed unless the completion
+	is unambiguous.  After a request by Stefan.  Minor cleanups in
+	ipy_stock_completers.
+
 2006-11-02  Ville Vainio  <[e-mail]>
 
 	* ipy_stock_completers.py: Add %run and %cd completers.