[patch] allow mmap take file offset as argument

[chrisl]

BPO	3765
Nosy	@terryjreedy

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2010-06-17.00:44:41.674>
created_at = <Date 2008-09-03.20:55:15.788>
labels = ['type-feature', 'library']
title = '[patch] allow mmap take file offset as argument'
updated_at = <Date 2010-06-17.00:44:41.672>
user = 'https://bugs.python.org/chrisl'

bugs.python.org fields:

activity = <Date 2010-06-17.00:44:41.672>
actor = 'terry.reedy'
assignee = 'none'
closed = True
closed_date = <Date 2010-06-17.00:44:41.674>
closer = 'terry.reedy'
components = ['Library (Lib)']
creation = <Date 2008-09-03.20:55:15.788>
creator = 'chrisl'
dependencies = []
files = []
hgrepos = []
issue_num = 3765
keywords = []
message_count = 2.0
messages = ['72416', '107971']
nosy_count = 2.0
nosy_names = ['terry.reedy', 'chrisl']
pr_nums = []
priority = 'normal'
resolution = 'out of date'
stage = None
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue3765'
versions = ['Python 2.5']

[chrisl]

The os.mmap function does not take file offset as
requirement. As a result, if python want to mmap
a piece of the file data at the very end of the
file, it needs to mmap the every thing before that.

Without offset argument, it is hard to work with
large files.

I make a patch to add the offset arguments to mmap
function call a while back. I wish it get included
in the future version of python.

http://mail.python.org/pipermail/python-list/2005-May/324213.html

Thanks

[terryjreedy]

This request is slightly confusing. The first sentence implies that you want offset to be a requirement (as opposed to being an option). That would not be acceptable. Later it seems that you want it added as an option (as opposed to not available). That seems sensible, and indeed, in 3.1, offset *is* an option (with a default of 0). So I am closing this as out-of-date (because fixed).

For future submissions, upload patches to the tracker as a file attached to the issue. The url you gave now gives 404 not found error.