Issue 9184: open() doc: default value for buffering is -1, not None (original) (raw)

Created on 2010-07-06 21:58 by vstinner, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
open_doc_buffering.patch vstinner,2010-07-06 21:58
Messages (5)
msg109434 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2010-07-06 21:58
Examples: * open(filename) works * open(filename, buffering=-1) works * open(filename, buffering=None) fails In the code: * _pyio.open(): default value is -1, type is int, no default value in the docstring * _io.open(): default value is not specified (C function), type is int, default value in the docstring is None In the doc: * open() default value is None The default value should be -1 everywhere. None is rejected by io.open (_io) and _pyio.open. Attached patch fixes _io.open() docstring Python documentation.
msg109975 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2010-07-11 09:33
Applied as r82801.
msg109981 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2010-07-11 10:15
Éric, I appreciate your wanting to help, but I don't think setting the stage after the fact makes much sense, especially since it is going away anyway...
msg109985 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2010-07-11 10:22
I think it's still better to set it correctly. Even assuming that it will go away, if the stage is set correctly it will be easier to convert it to something else if necessary (e.g. if the stage is left on "patch review" it might be turned to "needs review" even if it gets closed).
msg109987 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2010-07-11 10:29
Well, of course I don't want to make you unhappy, but don't ever expect me to make such a useless change myself :) Especially when an issue is closed, nobody is going to look at it again except by accident.
History
Date User Action Args
2022-04-11 14:57:03 admin set github: 53430
2010-07-11 10:29:06 georg.brandl set messages: +
2010-07-11 10:22:39 ezio.melotti set nosy: + ezio.melottimessages: +
2010-07-11 10:15:58 georg.brandl set messages: +
2010-07-11 09:42:15 eric.araujo set stage: resolved
2010-07-11 09:33:49 georg.brandl set status: open -> closednosy: + georg.brandlmessages: + resolution: accepted
2010-07-06 21:58:08 vstinner create