New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bpo-31368: Enhance os.preadv() documentation #7254
Conversation
Doc/library/os.rst
Outdated
resources. | ||
|
||
Currently, on Linux, this feature is usable only on a file descriptor opened | ||
using the O_DIRECT flag. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O_DIRECT should be written :data:O_DIRECT
to get a link to its doc.
leaving the file offset unchanged. *buffers* must be a sequence of | ||
:term:`bytes-like objects <bytes-like object>`. Buffers are processed in | ||
array order. Entire contents of the first buffer is written before | ||
proceeding to the second, and so on. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hum, a sentence like "The flags argument contains a bitwise OR of zero or more of the following flags:" is missing here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@vstinner Done! |
Doc/library/os.rst
Outdated
resources. | ||
|
||
Currently, on Linux, this feature is usable only on a file descriptor opened | ||
using the :data:`O_DIRECT`flag. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing space between ` and flag, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, sorry about that! Done
Thanks @pablogsal for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7. |
Thanks @pablogsal. I merged your PR and will backport it to 3.7. |
(cherry picked from commit 02e2a08) Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
GH-7258 is a backport of this pull request to the 3.7 branch. |
Based on this patch by @vstinner
https://bugs.python.org/issue31368