]> granicus.if.org Git - python/commitdiff
back in these go - thanks to Titus Brown for the fix
authorSkip Montanaro <skip@pobox.com>
Sat, 24 Nov 2007 14:30:47 +0000 (14:30 +0000)
committerSkip Montanaro <skip@pobox.com>
Sat, 24 Nov 2007 14:30:47 +0000 (14:30 +0000)
Lib/doctest.py
Lib/trace.py
Misc/NEWS

index e874a26817b1f3d0a7ffd1dbb24ab758aef11768..be62dad1c2b7ae8988396b4e43b503c0f587df0e 100644 (file)
@@ -320,8 +320,21 @@ class _OutputRedirectingPdb(pdb.Pdb):
     """
     def __init__(self, out):
         self.__out = out
+        self.__debugger_used = False
         pdb.Pdb.__init__(self, stdout=out)
 
+    def set_trace(self, frame=None):
+        self.__debugger_used = True
+        if frame is None:
+            frame = sys._getframe().f_back
+        pdb.Pdb.set_trace(self, frame)
+
+    def set_continue(self):
+        # Calling set_continue unconditionally would break unit test
+        # coverage reporting, as Bdb.set_continue calls sys.settrace(None).
+        if self.__debugger_used:
+            pdb.Pdb.set_continue(self)
+
     def trace_dispatch(self, *args):
         # Redirect stdout to the given stream.
         save_stdout = sys.stdout
index 364e3f7bc99ab4951a540308e42418c7e60d2317..3f006052544b43b25f862e8e3c5313400f341067 100644 (file)
@@ -286,6 +286,8 @@ class CoverageResults:
             # skip some "files" we don't care about...
             if filename == "<string>":
                 continue
+            if filename.startswith("<doctest "):
+                continue
 
             if filename.endswith((".pyc", ".pyo")):
                 filename = filename[:-1]
index 15fa9be11efb4f895875abf5127f108107734ac4..d0df601beb3479843972eaed50dc6dc052341226 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -290,6 +290,9 @@ Core and builtins
 Library
 -------
 
+- Issue 1429818: patch for trace and doctest modules so they play nicely
+  together.
+
 - doctest made a bad assumption that a package's __loader__.get_data()
   method used universal newlines.