- Work around pexcept buglet which causes wraparound problems with long

input lines in irunner.

- Move arg_split to genutils so it can be more generally reused.

- Modify OInspect to expose a publicly available getsource() routine.  At
the request of SAGE, which knows how to extract docstrings from Pyrex
extensions.

IPython/OInspect.py

@@ -6,7 +6,7 b' Uses syntax highlighting for presenting the various information elements.'
 Similar in spirit to the inspect module, but all calls take a name argument to
 reference the name under which an object is being read.
 
-$Id: OInspect.py 1625 2006-08-12 10:34:44Z vivainio $
+$Id: OInspect.py 1850 2006-10-28 19:48:13Z fptest $
 """
 
 #*****************************************************************************
@@ -136,6 +136,27 b' def getdoc(obj):'
             ds = '%s\n%s' % (ds,ds2)
     return ds
 
+def getsource(obj,is_binary=False):
+    """Wrapper around inspect.getsource.
+
+    This can be modified by other projects to provide customized source
+    extraction.
+
+    Inputs:
+
+    - obj: an object whose source code we will attempt to extract.
+
+    Optional inputs:
+
+    - is_binary: whether the object is known to come from a binary source.
+    This implementation will skip returning any output for binary objects, but
+    custom extractors may know how to meaninfully process them."""
+    
+    if is_binary:
+        return None
+    else:
+        return inspect.getsource(obj)
+
 #****************************************************************************
 # Class definitions
 
@@ -262,7 +283,7 b' class Inspector:'
         # Flush the source cache because inspect can return out-of-date source
         linecache.checkcache()
         try:
-            src = inspect.getsource(obj) 
+            src = getsource(obj) 
         except:
             self.noinfo('source',oname)
         else:
@@ -402,15 +423,16 b' class Inspector:'
             linecache.checkcache()
             source_success = False
             try:
-                if not binary_file:
-                    source = self.format(inspect.getsource(obj))
+                source = self.format(getsource(obj,binary_file))
+                if source:
                     out.write(header('Source:\n')+source.rstrip())
                     source_success = True
-            except:
+            except Exception, msg:
                 pass
             
             if ds and not source_success:
-                out.writeln(header('Docstring [source file open failed]:\n') + indent(ds))
+                out.writeln(header('Docstring [source file open failed]:\n')
+                            + indent(ds))
 
         # Constructor docstring for classes
         if obj_type is types.ClassType:

IPython/Prompts.py

@@ -2,7 +2,7 b''
 """
 Classes for handling input/output prompts.
 
-$Id: Prompts.py 1366 2006-06-15 19:45:50Z vivainio $"""
+$Id: Prompts.py 1850 2006-10-28 19:48:13Z fptest $"""
 
 #*****************************************************************************
 #       Copyright (C) 2001-2006 Fernando Perez <[e-mail]>

IPython/iplib.py

@@ -6,7 +6,7 b' Requires Python 2.3 or newer.'
 
 This file contains all the classes and helper functions specific to IPython.
 
-$Id: iplib.py 1828 2006-10-16 02:04:33Z fptest $
+$Id: iplib.py 1850 2006-10-28 19:48:13Z fptest $
 """
 
 #*****************************************************************************
@@ -665,10 +665,12 b' class InteractiveShell(object,Magic):'
         except AttributeError:
             pass
 
-        # I don't like assigning globally to sys, because it means when embedding
-        # instances, each embedded instance overrides the previous choice. But
-        # sys.displayhook seems to be called internally by exec, so I don't see a
-        # way around it.
+        # I don't like assigning globally to sys, because it means when
+        # embedding instances, each embedded instance overrides the previous
+        # choice. But sys.displayhook seems to be called internally by exec,
+        # so I don't see a way around it.  We first save the original and then
+        # overwrite it.
+        self.sys_displayhook = sys.displayhook
         sys.displayhook = self.outputcache
 
         # Set user colors (don't do it in the constructor above so that it

IPython/irunner.py

@@ -137,6 +137,14 b' class InteractiveRunner(object):'
         c = pexpect.spawn(self.program,self.args,timeout=None)
         c.delaybeforesend = self.delaybeforesend
             
+        # pexpect hard-codes the terminal size as (24,80) (rows,columns).
+        # This causes problems because any line longer than 80 characters gets
+        # completely overwrapped on the printed outptut (even though
+        # internally the code runs fine).  We reset this to 99 rows X 200
+        # columns (arbitrarily chosen), which should avoid problems in all
+        # reasonable cases.
+        c.setwinsize(99,200)
+            
         prompts = c.compile_pattern_list(self.prompts)
 
         prompt_idx = c.expect_list(prompts)

doc/ChangeLog

@@ -1,3 +1,15 b''
[phone]  Fernando Perez  <[e-mail]>
+
+	* IPython/genutils.py (arg_split): moved to genutils, since it's a
+	pretty generic function and useful for other things.
+
+	* IPython/OInspect.py (getsource): Add customizable source
+	extractor.  After a request/patch form W. Stein (SAGE).
+
+	* IPython/irunner.py (InteractiveRunner.run_source): reset tty
+	window size to a more reasonable value from what pexpect does,
+	since their choice causes wrapping bugs with long input lines.
+
 2006-10-28  Ville Vainio  <[e-mail]>
 
 	* Magic.py (%run): Save and restore the readline history from