summaryrefslogtreecommitdiff
path: root/coverage/collector.py
diff options
context:
space:
mode:
authorNed Batchelder <nedbat@gmail.com>2013-09-16 09:13:21 -0400
committerNed Batchelder <nedbat@gmail.com>2013-09-16 09:13:21 -0400
commitd39438e5121ebddaf8aa0f6b2abdf7ce07f7ea31 (patch)
tree1ee0cefdf520871cc54583410d8525e4dbf2eed8 /coverage/collector.py
parentdfe83e5fcdb5cc48880fda91d3d78353cb6ce4f7 (diff)
parentf5675121528fe9d4729f0ba8052282de99ec9b79 (diff)
downloadpython-coveragepy-d39438e5121ebddaf8aa0f6b2abdf7ce07f7ea31.tar.gz
Merged in rogerjhu/coverage.py (pull request #19)
Make UTF-8 detection more robust.
Diffstat (limited to 'coverage/collector.py')
-rw-r--r--coverage/collector.py9
1 files changed, 5 insertions, 4 deletions
diff --git a/coverage/collector.py b/coverage/collector.py
index 781a0fa..9a74700 100644
--- a/coverage/collector.py
+++ b/coverage/collector.py
@@ -75,10 +75,11 @@ class PyTracer(object):
# in this file.
self.data_stack.append((self.cur_file_data, self.last_line))
filename = frame.f_code.co_filename
- tracename = self.should_trace_cache.get(filename)
- if tracename is None:
+ if filename not in self.should_trace_cache:
tracename = self.should_trace(filename, frame)
self.should_trace_cache[filename] = tracename
+ else:
+ tracename = self.should_trace_cache[filename]
#print("called, stack is %d deep, tracename is %r" % (
# len(self.data_stack), tracename))
if tracename:
@@ -166,7 +167,7 @@ class Collector(object):
"""Create a collector.
`should_trace` is a function, taking a filename, and returning a
- canonicalized filename, or False depending on whether the file should
+ canonicalized filename, or None depending on whether the file should
be traced or not.
If `timid` is true, then a slower simpler trace function will be
@@ -210,7 +211,7 @@ class Collector(object):
# A cache of the results from should_trace, the decision about whether
# to trace execution in a file. A dict of filename to (filename or
- # False).
+ # None).
self.should_trace_cache = {}
# Our active Tracers.