bpo-37471: mmap module adding FreeBSD specific flag into the constants #14513
Conversation
ghost
changed the title
ghost
changed the title
vstinner
left a comment
vstinner
left a comment
There was a problem hiding this comment.
Would it be possible to document the constant in https://docs.python.org/dev/library/mmap.html ?
It seems like other MAP_xxx constants are not documented yet and so should be documented as well.
|
They are (shortly) in the flags section. Ok for adding the new constant too. |
|
The minimum doc should be a list of constants, like: https://docs.python.org/dev/library/os.html#os.O_RDONLY |
|
Fair enough ! ought to be before madvise's I think. |
Doc/library/mmap.rst
Outdated
There was a problem hiding this comment.
| pages of 2MB (x86) or so, it is enabled by default, at boot time, | |
| pages of 2 MiB (x86) or so, it is enabled by default, at boot time, |
Is it x86 (32-bit) or x86-64 (64-bit)?
There was a problem hiding this comment.
Write maybe (x86, x86-64) in this case... But I'm not sure if Python documentation should go too far in the description, since FreeBSD may decide to add support for MAP_ALIGNED_SUPER for other platforms.
There was a problem hiding this comment.
It supports more than x86 already in fact but you re right I ll try to find more concise wording
There was a problem hiding this comment.
| Adding new 'MAP_ALIGNED_SUPER' FreeBSD constant. | |
| Adding new :data:`mmap.MAP_ALIGNED_SUPER` FreeBSD constant. |
Doc/library/mmap.rst
Outdated
There was a problem hiding this comment.
You have to document that MAP_ALIGNED_SUPER is new, example:
.. versionadded:: 3.9
Add :data:`MAP_ALIGNED_SUPER` constant.
Doc/library/mmap.rst
Outdated
There was a problem hiding this comment.
"MAP_* constants" is not really a sentence. Maybe move it after the default, and write something like:
The default value is :const:`MAP_SHARED`.
See :ref:`MAP_* constants <mmap-constants>`.
Doc/library/mmap.rst
Outdated
There was a problem hiding this comment.
Can you move this to the your new MAP_* Constants section?
There was a problem hiding this comment.
Can you try to merge the 2 changelog entries? Put content of the 2nd entry in the first one, remove the second file.
Doc/library/mmap.rst
Outdated
There was a problem hiding this comment.
You have to document that MAP_CONCEAL is new.
Doc/library/mmap.rst
Outdated
There was a problem hiding this comment.
I don't understand this sentence :-/ I suggest:
:const:`MAP_ALIGNED_SUPER` (FreeBSD only) uses super pages feature,
it is enabled by default in most of architectures and exists since 2008.
3 participants
https://bugs.python.org/issue37471