Giampaolo Rodola' <g.rod...@gmail.com> added the comment: > Copy *count* bytes from file descriptor *in* to file descriptor > *out*, starting at *offset* and continuing for *count* bytes.
The latter part is incorrect as it is not guaranteed that all bytes specified in "count" argument are going to be sent and it also sounds like the function is blocking. I'd change that in just "Copy *count* bytes from file descriptor *in* to file descriptor *out*." Also, I'd be for using the BSD notation and rename the "count" argument to "nbytes", which is more clear. Docstring should be changed to reflect the keyword arguments: - sendfile(out, in, offset, count, headers, trailers, flags)\n\ + sendfile(out, in, offset, count[, headers[, trailers], flags]]])\n\ Finally, tests for header and trailer arguments should be written. The rest of the patch looks ok to me. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue10882> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
