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.

classification
Title: proposed 3000 patch for socket.py - "socket GC worries"
Type: Stage:
Components: Library (Lib) Versions: Python 3.0
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: christian.heimes Nosy List: christian.heimes, gvanrossum, janssen
Priority: normal Keywords: patch

Created on 2007-11-13 19:36 by janssen, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
b.diff janssen, 2007-11-13 19:36 Patch
Messages (6)
msg57470 - (view) Author: Bill Janssen (janssen) * (Python committer) Date: 2007-11-13 19:36
This patch essentially makes GC of sockets work again.

See http://mail.python.org/pipermail/python-3000/2007-October/
011058.html and all the threads in http://mail.python.org/pipermail/
python-3000/2007-October/thread.html with subject line "socket GC 
worries" for a full discussion.
msg57493 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2007-11-14 18:22
Crys, can you test this on Windows and if it does, assign back to me?
msg57503 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2007-11-14 19:28
Hi Janssen!

The patch at http://bugs.python.org/issue1378 adds suport for
_socket.socket.dup() on Windows. It can't dup file descriptors but it
can dup socket handlers. Would it make your patch easier?
msg57507 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2007-11-14 20:35
I've checked the patch on Windows. I've run the regression tests with
-R:: for test_urllib2 test_urllib2net test_urllib test_urllibnet
test_socket. The tests are passing and no ref leak shows up.
msg57509 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2007-11-14 22:32
Committed revision 58970 (with slight changes).

Bill, you can now submit your SSL code for review.
msg57518 - (view) Author: Bill Janssen (janssen) * (Python committer) Date: 2007-11-15 02:59
Once I've tried it again.

Bill

On Nov 14, 2007 2:32 PM, Guido van Rossum <report@bugs.python.org> wrote:
>
> Guido van Rossum added the comment:
>
> Committed revision 58970 (with slight changes).
>
> Bill, you can now submit your SSL code for review.
>
> ----------
> assignee: gvanrossum -> tiran
> resolution:  -> accepted
> status: open -> closed
>
>
> __________________________________
> Tracker <report@bugs.python.org>
> <http://bugs.python.org/issue1439>
> __________________________________
>
History
Date User Action Args
2022-04-11 14:56:28adminsetgithub: 45780
2008-01-06 22:29:45adminsetkeywords: - py3k
versions: Python 3.0
2007-11-15 02:59:00janssensetmessages: + msg57518
2007-11-14 22:32:38gvanrossumsetstatus: open -> closed
assignee: gvanrossum -> christian.heimes
resolution: accepted
messages: + msg57509
2007-11-14 20:35:28christian.heimessetassignee: christian.heimes -> gvanrossum
messages: + msg57507
2007-11-14 19:28:12christian.heimessetmessages: + msg57503
2007-11-14 18:22:48gvanrossumsetassignee: gvanrossum -> christian.heimes
messages: + msg57493
nosy: + christian.heimes
2007-11-13 19:36:37janssencreate