Message205084
The patch is fine, but it is hard to rely on it to prevent bugs from happening because that requires cooperation from all modules registering signal handlers.
Anyway it facilitates reusing code that was not written for an event-driven context (and many will do that through .run_in_executor()). If the patch is accepted, it would be wise to write a note in .run_in_executor()'s doc saying that asyncio uses SA_RESTART by default in all its handler and that EINTR is prevented *as long as* no other handlers are registered elsewhere. |
|
Date |
User |
Action |
Args |
2013-12-03 08:36:41 | aba | set | recipients:
+ aba, gvanrossum, gregory.p.smith, vstinner, neologix |
2013-12-03 08:36:41 | aba | set | messageid: <1386059801.4.0.872118194562.issue19850@psf.upfronthosting.co.za> |
2013-12-03 08:36:41 | aba | link | issue19850 messages |
2013-12-03 08:36:40 | aba | create | |
|