[issue21435] Segfault in gc with cyclic trash

2014-05-10 Thread Andrew Svetlov

Andrew Svetlov added the comment:

@Tim nothing to close, aiohttp is separate library based on asyncio.
It just uses constructions like self.attr = None in __del__

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-09 Thread Andrew Svetlov

Andrew Svetlov added the comment:

Thanks a lot!
The patch fixes crush dump issue with __del__ in aiohttp library tests also.

--
nosy: +asvetlov

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-09 Thread Tim Peters

Tim Peters added the comment:

@asvetlov, glad this fixes crashes in aiohttp library tests too, but I hadn't 
heard about that before.  Is there an open bug report about it on this tracker 
(so we can close it)?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-09 Thread Peter Inglesby

Peter Inglesby added the comment:

It was actually through playing with aiohttp that I first hit this issue.  I 
think I originally hit the problem with something like:

import asyncio
import aiohttp

@asyncio.coroutine
def do_work(future):
response = yield from aiohttp.request('get', 'http://google.com')
future.set_result(None)

loop = asyncio.get_event_loop()
future = asyncio.Future()
asyncio.Task(do_work(future))
future.add_done_callback(print)
loop.run_forever()

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-08 Thread Tim Peters

Changes by Tim Peters t...@python.org:


--
title: Segfault with cyclic reference and asyncio.Future - Segfault in gc with 
cyclic trash

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-08 Thread Tim Peters

Tim Peters added the comment:

finalize42.patch includes a test case.  If nobody objects within a few hours, 
I'll commit it.

--
Added file: http://bugs.python.org/file35187/finalize42.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 64ba3f2de99c by Tim Peters in branch '3.4':
Issue #21435: Segfault in gc with cyclic trash
http://hg.python.org/cpython/rev/64ba3f2de99c

New changeset cb9a3985df00 by Tim Peters in branch 'default':
Merge from 3.4.
http://hg.python.org/cpython/rev/cb9a3985df00

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue21435] Segfault in gc with cyclic trash

2014-05-08 Thread Tim Peters

Changes by Tim Peters t...@python.org:


--
resolution:  - fixed
stage:  - resolved
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21435
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com