]> granicus.if.org Git - python/commitdiff
Issue #4050: inspect.findsource/getsource now raise an IOError if the 'source'
authorR. David Murray <rdmurray@bitdance.com>
Wed, 13 May 2009 17:14:11 +0000 (17:14 +0000)
committerR. David Murray <rdmurray@bitdance.com>
Wed, 13 May 2009 17:14:11 +0000 (17:14 +0000)
file is a binary.  Patch by Brodie Rao, test by Daniel Diniz.

Lib/inspect.py
Lib/test/test_inspect.py
Misc/ACKS
Misc/NEWS

index 1453f3b7cf715f8cc6c0c1be81b191b2ec82a8f1..3305c8d0400782dc53bbf9dcc47b964617cf2ab2 100644 (file)
@@ -523,7 +523,9 @@ def findsource(object):
     or code object.  The source code is returned as a list of all the lines
     in the file and the line number indexes a line in that list.  An IOError
     is raised if the source code cannot be retrieved."""
-    file = getsourcefile(object) or getfile(object)
+    file = getsourcefile(object)
+    if not file:
+        raise IOError('source code not available')
     module = getmodule(object, file)
     if module:
         lines = linecache.getlines(file, module.__dict__)
index 7312cac0080f65ebbd71c541db50dceed9801082..5c414336130c7ecb6fccc64e1017a6ed144513e2 100644 (file)
@@ -9,6 +9,9 @@ from test.test_support import TESTFN, run_unittest
 from test import inspect_fodder as mod
 from test import inspect_fodder2 as mod2
 
+# C module for test_findsource_binary
+import time
+
 # Functions tested in this suite:
 # ismodule, isclass, ismethod, isfunction, istraceback, isframe, iscode,
 # isbuiltin, isroutine, isgenerator, isgeneratorfunction, getmembers,
@@ -329,6 +332,10 @@ class TestBuggyCases(GetSourceBase):
     def test_method_in_dynamic_class(self):
         self.assertSourceEqual(mod2.method_in_dynamic_class, 95, 97)
 
+    def test_findsource_binary(self):
+        self.assertRaises(IOError, inspect.getsource, time)
+        self.assertRaises(IOError, inspect.findsource, time)
+
 # Helper for testing classify_class_attrs.
 def attrs_wo_objs(cls):
     return [t[:3] for t in inspect.classify_class_attrs(cls)]
index 9db9f61b8e20a3c53a6ff782339d0271b2bb9832..5cae037fac4b253e49c48a17df1f501998290b97 100644 (file)
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -171,6 +171,7 @@ Raghuram Devarakonda
 Toby Dickenson
 Mark Dickinson
 Jack Diederich
+Daniel Diniz
 Yves Dionne
 Daniel Dittmar
 Jaromir Dolecek
@@ -577,6 +578,7 @@ Eduardo P
 Brian Quinlan
 Anders Qvist
 Burton Radons
+Brodie Rao
 Antti Rasinen
 Eric Raymond
 Edward K. Ream
index b717729458aee5e900e889456abe1bf2aa8712c4..2203621b479785b15dcbba79d394bc7ed9161f94 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -293,6 +293,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #4050: inspect.findsource/getsource now raise an IOError if the 'source'
+  file is a binary.  Patch by Brodie Rao, tests by Daniel Diniz.
+
 - Issue #5977: distutils build_ext.get_outputs was not taking into account the
   inplace option. Initial patch by kxroberto.