This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author neologix
Recipients Daniel.Evers, exarkun, neologix, pitrou, tarek
Date 2011-05-05.18:53:48
SpamBayes Score 3.9362957e-13
Marked as misclassified No
Message-id <1304621629.04.0.384700271522.issue8498@psf.upfronthosting.co.za>
In-reply-to
Content
> Thanks for the tip. I added the unit test and uploaded my final patch
> (which includes all changes).

A couple comments (note that I'm not entitled to accept or commit a patch, so feel free to ignore them if I'm just being a pain):
- "the maximum value is system-dependent (usually 5)"
I would remove this "usually 5" part: since the kernel silently adjusts the value, it shouldn't be of interest to the programmer. Furthermore, this was probably true a decade ago, but nowadays it doesn't hold anymore, Linux for example allows a much higher limit
- port = support.bind_port(srv)
Since you don't care about which port you get bound to, you could discard the port value, or even call srv.bind((HOST, 0)) directly
- I'm not sure that creating a new SocketListenBacklog0Test TestCase just for this check is necessary. Since it's so short, you could just add a new test to e.g. GeneralModuleTests

> Is it ok to remove the files I uploaded previously? 

It's probably a good idea, so there won't be any confusion with "stale" patches and test scripts.
History
Date User Action Args
2011-05-05 18:53:49neologixsetrecipients: + neologix, exarkun, pitrou, tarek, Daniel.Evers
2011-05-05 18:53:49neologixsetmessageid: <1304621629.04.0.384700271522.issue8498@psf.upfronthosting.co.za>
2011-05-05 18:53:48neologixlinkissue8498 messages
2011-05-05 18:53:48neologixcreate