summaryrefslogtreecommitdiff
path: root/coverage/report.py
diff options
context:
space:
mode:
authorNed Batchelder <ned@nedbatchelder.com>2015-09-22 12:56:00 -0400
committerNed Batchelder <ned@nedbatchelder.com>2015-09-22 12:56:00 -0400
commitecc43a09f99105b11bc428fa9d616733950ce524 (patch)
tree4f69ff80e586efdf56d50cf30774a0d875d38f82 /coverage/report.py
parenta08763effeb774d4a13f1301846f7a23b9d4ff3b (diff)
downloadpython-coveragepy-ecc43a09f99105b11bc428fa9d616733950ce524.tar.gz
Use more compact code for includes and omits
Diffstat (limited to 'coverage/report.py')
-rw-r--r--coverage/report.py24
1 files changed, 7 insertions, 17 deletions
diff --git a/coverage/report.py b/coverage/report.py
index f153aa5..78ad748 100644
--- a/coverage/report.py
+++ b/coverage/report.py
@@ -35,27 +35,17 @@ class Reporter(object):
`morfs` is a list of modules or file names.
"""
- self.file_reporters = self.coverage._get_file_reporters(morfs)
+ reporters = self.coverage._get_file_reporters(morfs)
if self.config.include:
- patterns = prep_patterns(self.config.include)
- matcher = FnmatchMatcher(patterns)
- filtered = []
- for fr in self.file_reporters:
- if matcher.match(fr.filename):
- filtered.append(fr)
- self.file_reporters = filtered
+ matcher = FnmatchMatcher(prep_patterns(self.config.include))
+ reporters = [fr for fr in reporters if matcher.match(fr.filename)]
if self.config.omit:
- patterns = prep_patterns(self.config.omit)
- matcher = FnmatchMatcher(patterns)
- filtered = []
- for fr in self.file_reporters:
- if not matcher.match(fr.filename):
- filtered.append(fr)
- self.file_reporters = filtered
-
- self.file_reporters.sort()
+ matcher = FnmatchMatcher(prep_patterns(self.config.omit))
+ reporters = [fr for fr in reporters if not matcher.match(fr.filename)]
+
+ self.file_reporters = sorted(reporters)
def report_files(self, report_fn, morfs, directory=None):
"""Run a reporting function on a number of morfs.