[issue10705] HTTPConnection.set_debuglevel has no information about level range

2010-12-14 Thread R. David Murray
R. David Murray added the comment: I've committed a fix in r87256. I looked at the code some more and tried to be a precise as possible without getting too wordy. (The fix will get backported by and by.) -- resolution: -> fixed stage: -> committed/rejected status: open -> closed

[issue10705] HTTPConnection.set_debuglevel has no information about level range

2010-12-14 Thread R. David Murray
R. David Murray added the comment: I think a PEP is a bit of overkill for a small doc update :) A patch would be fine...but I can also just make the change. I'll probably tweak your wording a bit. -- ___ Python tracker

[issue10705] HTTPConnection.set_debuglevel has no information about level range

2010-12-14 Thread J_Tom_Moon_79
J_Tom_Moon_79 added the comment: > Care to propose a patch? oops. I just found http://www.python.org/dev/patches/ I'll get around to creating a PEP sometime soon. -- ___ Python tracker ___

[issue10705] HTTPConnection.set_debuglevel has no information about level range

2010-12-14 Thread J_Tom_Moon_79
J_Tom_Moon_79 added the comment: Hi David, Currently the 3.1 documentation reads: """HTTPConnection.set_debuglevel(level)ΒΆ Set the debugging level (the amount of debugging output printed). The default debug level is 0, meaning no debugging output is printed.""" How about: """HTTPConnection.

[issue10705] HTTPConnection.set_debuglevel has no information about level range

2010-12-14 Thread R. David Murray
R. David Murray added the comment: Or, since this is Python, they could look at the code and find out that all levels above zero are equivalent. (If I had to guess I'd say 'level' was either future proofing or designed for the use of subclasses). But you are right, this should be mentioned i

[issue10705] HTTPConnection.set_debuglevel has no information about level range

2010-12-14 Thread J_Tom_Moon_79
New submission from J_Tom_Moon_79 : Function HTTPConnection.set_debuglevel(level) There is no range of acceptable debug levels. What should be the difference in set_debuglevel(1) and set_debuglevel(1000)? If the documentation lists the levels this would save users from needing to do tedious tria