diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2015-09-22 12:56:00 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2015-09-22 12:56:00 -0400 |
commit | ecc43a09f99105b11bc428fa9d616733950ce524 (patch) | |
tree | 4f69ff80e586efdf56d50cf30774a0d875d38f82 /coverage/report.py | |
parent | a08763effeb774d4a13f1301846f7a23b9d4ff3b (diff) | |
download | python-coveragepy-ecc43a09f99105b11bc428fa9d616733950ce524.tar.gz |
Use more compact code for includes and omits
Diffstat (limited to 'coverage/report.py')
-rw-r--r-- | coverage/report.py | 24 |
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. |