Kaydet (Commit) ccd9e75b authored tarafından Barry Warsaw's avatar Barry Warsaw

test_both(): I believe this was a typo: m is only defined if no

exception occurred so it should only be closed in the else clause.
Without this change we can an UnboundLocalError on Linux:

Traceback (most recent call last):
  File "Lib/test/test_mmap.py", line 304, in ?
    test_both()
  File "Lib/test/test_mmap.py", line 208, in test_both
    m.close()
UnboundLocalError: local variable 'm' referenced before assignment
üst 1a5b9562
......@@ -205,7 +205,7 @@ def test_both():
# we expect a ValueError on Unix, but not on Windows
if not sys.platform.startswith('win'):
verify(0, "Opening mmap with size+1 should raise ValueError.")
m.close()
m.close()
f.close()
if sys.platform.startswith('win'):
# Repair damage from the resizing test.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment