diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2014-05-20 07:23:56 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2014-05-20 07:23:56 -0400 |
commit | 1d61f62a1ca690265359feda96cb1a9398245060 (patch) | |
tree | fbbe8d01d4b70e4ed3887a2fb73900bf0c70c487 /coverage/codeunit.py | |
parent | 7a0e7fec3fc29c049c59483dbc6f6334e0a90535 (diff) | |
download | python-coveragepy-1d61f62a1ca690265359feda96cb1a9398245060.tar.gz |
Clean up codeunit, as usual
Diffstat (limited to 'coverage/codeunit.py')
-rw-r--r-- | coverage/codeunit.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/coverage/codeunit.py b/coverage/codeunit.py index 4fa9cf8..179de01 100644 --- a/coverage/codeunit.py +++ b/coverage/codeunit.py @@ -1,6 +1,6 @@ """Code unit (module) handling for Coverage.""" -import os, re +import os from coverage.backward import open_python_source, string_class from coverage.misc import CoverageException, NoSource @@ -121,7 +121,8 @@ class CodeUnit(object): """Return the source code, as a string.""" if os.path.exists(self.filename): # A regular text file: open it. - return open_python_source(self.filename).read() + with open_python_source(self.filename) as f: + return f.read() # Maybe it's in a zip file? source = self.file_locator.get_zip_data(self.filename) @@ -231,9 +232,8 @@ class MakoParser(CodeParser): def parse_source(self): """Returns executable_line_numbers, excluded_line_numbers""" - r = set(self.metadata['line_map'].values()) - print r - return r, set() + executable = set(self.metadata['line_map'].values()) + return executable, set() def translate_lines(self, lines): tlines = set(self.metadata['full_line_map'].get(l, -1) for l in lines) |