Maxim Yanchenko <[email protected]> added the comment:
Well, "n+1" is clearly outside the file, wile "n" is within and therefore valid.
Also, if your position is to forbid zero-size mmapping completely, then the
checks inside "if (map_size == 0) {" don't make any sense, especially as they
may or may fail.
>From the existing code, zero-size mmapping is OK as long as offset is OK, so
>the question is whether we consider offset pointing to the end of the file OK.
>To me, it's fine and valid, and there are valid cases like NumPy's zero-size
>arrays, hence the proposed patch.
Removing the check completely is a viable option too, it was already requested
for special files:
http://bugs.python.org/issue12556
I believe users should have an ability to enjoy whatever their OS provides, and
"deal with the consequences" as you said.
----------
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue13148>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
