[Patches] [ python-Patches-477161 ] New "access" keyword for mmap

[noreply@sourceforge.net]

Patches item #477161, was opened at 2001-11-01 07:12
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=477161&group_id=5470

Category: Modules
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Jay T Miller (jaytmiller)
Assigned to: Nobody/Anonymous (nobody)
>Summary: New "access" keyword for mmap

Initial Comment:
The attached patch adds a new keyword parameter to mmap
which provides a platform independent way of asking for
3 basic kinds of mmaps:   readonly, write-through, and
copy-on-write.  For now, platform independent means
UNIX and Win32.

A readonly mmap creates a true readonly section of the
process memory map, but python level access is guarded
so that attempts to write the section (or fetch a
writeable buffer) raise a ValueError exception.  A
readonly mmap is not resizeable.

A write-through memory map is both readable and
writeable, and updates to the memory are reflected onto
the underlying file.

A copy-on-write memory map is both readable and
writeable, but updates to the memory are *not*
reflected on the underlying file.  A copy-on-write mmap
is not resizeable.

The patch modifies mmapodule.c,  libmmap.tex, and
test_mmap.py.



----------------------------------------------------------------------

>Comment By: Martin v. Löwis (loewis)
Date: 2001-11-02 16:09

Message:
Logged In: YES 
user_id=21627

Since this patch introduces a significant new feature, I'd 
recommend to postpone it until after 2.2.

The patch itself looks ok, but the following improvements 
should be made:
- indentation is incorrect, please see PEP 7.
- the documentation has some typos, ACCES_ -> ACCESS_
- if possible, the documentation should indicate that
  the additional parameters are new to Python 2.3.


----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=477161&group_id=5470