diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2009-04-16 22:46:30 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2009-04-16 22:46:30 -0400 |
commit | 0b38c15b0bae25ccd53d6f59aa7760d6b612e184 (patch) | |
tree | 31f76f129f43c29119e4ca419ffb72e0fb1fe550 /coverage/data.py | |
parent | 4df3f70cb7c5b97d035d82b506a9a0b9e0f81fa9 (diff) | |
download | python-coveragepy-git-0b38c15b0bae25ccd53d6f59aa7760d6b612e184.tar.gz |
Remove an unused parameter, and other small cleanups to data storage
Diffstat (limited to 'coverage/data.py')
-rw-r--r-- | coverage/data.py | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/coverage/data.py b/coverage/data.py index 06f46343..deebfbe3 100644 --- a/coverage/data.py +++ b/coverage/data.py @@ -26,10 +26,8 @@ class CoverageData: # self.executed = {} - def usefile(self, use_file=True, filename_default=None): + def usefile(self, use_file=True): self.use_file = use_file - if filename_default and not self.filename: - self.filename_default = filename_default def read(self, parallel=False): """Read coverage data from the coverage data file (if it exists).""" @@ -69,8 +67,10 @@ class CoverageData: """ try: fdata = open(filename, 'rb') - executed = pickle.load(fdata) - fdata.close() + try: + executed = pickle.load(fdata) + finally: + fdata.close() if isinstance(executed, types.DictType): return executed else: @@ -94,8 +94,8 @@ class CoverageData: for filename, file_data in new_data.items(): self.executed.setdefault(filename, {}).update(file_data) - def add_raw_data(self, data_points): - """Add raw data. + def add_line_data(self, data_points): + """Add executed line data. `data_points` is (filename, lineno) pairs. |