Commit 50dc65f6 authored by Antoine Pitrou's avatar Antoine Pitrou

Merged revisions 88036 via svnmerge from

svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r88036 | antoine.pitrou | 2011-01-15 18:25:58 +0100 (sam., 15 janv. 2011) | 3 lines

  Fix mmap and test_mmap under Windows too (followup to r88022)
........
parent fb7bc3d2
...@@ -329,12 +329,16 @@ class MmapTests(unittest.TestCase): ...@@ -329,12 +329,16 @@ class MmapTests(unittest.TestCase):
# map length with an offset doesn't cause a segfault. # map length with an offset doesn't cause a segfault.
if not hasattr(os, "stat"): if not hasattr(os, "stat"):
self.skipTest("needs os.stat") self.skipTest("needs os.stat")
with open(TESTFN, "wb+") as f: # NOTE: allocation granularity is currently 65536 under Win64,
f.write(49152 * b'm') # Arbitrary character # and therefore the minimum offset alignment.
with open(TESTFN, "wb") as f:
f.write((65536 * 2) * b'm') # Arbitrary character
with open(TESTFN, "rb") as f: with open(TESTFN, "rb") as f:
mf = mmap.mmap(f.fileno(), 0, offset=40960, access=mmap.ACCESS_READ) mf = mmap.mmap(f.fileno(), 0, offset=65536, access=mmap.ACCESS_READ)
self.assertRaises(IndexError, mf.__getitem__, 45000) try:
self.assertRaises(IndexError, mf.__getitem__, 80000)
finally:
mf.close() mf.close()
def test_move(self): def test_move(self):
......
...@@ -1269,6 +1269,7 @@ new_mmap_object(PyTypeObject *type, PyObject *args, PyObject *kwdict) ...@@ -1269,6 +1269,7 @@ new_mmap_object(PyTypeObject *type, PyObject *args, PyObject *kwdict)
else else
m_obj->size = low; m_obj->size = low;
#endif #endif
m_obj->size -= offset;
} else { } else {
m_obj->size = map_size; m_obj->size = map_size;
} }
......
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