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.
É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...
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).
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.