[Python-Dev] Adding timeout to socket.py and httplib.py (original) (raw)

Facundo Batista facundo at taniquetil.com.ar
Tue Mar 20 20:24:30 CET 2007


Alan Kennedy wrote:

[Facundo]

But, I recognize that maybe it's [connect] not the best name. What about "createconnection"? I have no strong feelings about it, other than to say it should not be "connect". How about

Ok. "create_connection", then.

Ah, but it's too late by the time the socket.connect call returns: the timeout/blocking behaviour of the socket.connect call is the very thing we're trying to control.

It's not the very thing, just one of them... whatever, you have a point.

Whereas with the slightly modified API I suggested earlier, it simply becomes

I'm OK with that API, except that you're losing position parameters. It's OK to always need to put the "timeout="?

The problem here is that I used None to check if you passed a parameter or not, an idiom well stablished in Python, but in this very case None has a meaning for itself.

I'm +0 on having the obligation to a named parameter here.

So, I have two modifications to make to the patch:

Is everybody ok with this?

Regards,

-- . Facundo . Blog: http://www.taniquetil.com.ar/plog/ PyAr: http://www.python.org/ar/



More information about the Python-Dev mailing list