diff options
author | Brett Cannon <brett@python.org> | 2011-01-28 13:53:34 -0800 |
---|---|---|
committer | Brett Cannon <brett@python.org> | 2011-01-28 13:53:34 -0800 |
commit | 7239baf097df53b1da81854452fe027855ca2b9c (patch) | |
tree | 4741c241d45705f5de1a728e97999e92294609b6 /coverage/html.py | |
parent | 9681a8f11e84ae57abc7cbfba807ca847884e5a5 (diff) | |
download | python-coveragepy-7239baf097df53b1da81854452fe027855ca2b9c.tar.gz |
Properly close files.
Diffstat (limited to 'coverage/html.py')
-rw-r--r-- | coverage/html.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/coverage/html.py b/coverage/html.py index 76e2890..b68cc93 100644 --- a/coverage/html.py +++ b/coverage/html.py @@ -18,7 +18,11 @@ def data_filename(fname): def data(fname): """Return the contents of a data file of ours.""" - return open(data_filename(fname)).read() + data_file = open(data_filename(fname)) + try: + return data_file.read() + finally: + data_file.close() class HtmlReporter(Reporter): @@ -68,8 +72,11 @@ class HtmlReporter(Reporter): def html_file(self, cu, analysis): """Generate an HTML file for one source file.""" - - source = cu.source_file().read() + source_file = cu.source_file() + try: + source = source_file.read() + finally: + source_file.close() nums = analysis.numbers |