[issue1564547] Py_signal_pipe

Changes by Guido van Rossum: -- resolution: accepted -> rejected superseder: -> Patch for signal.set_wakeup_fd _ Tracker <[EMAIL PROTECTED]> _ _

[issue1564547] Py_signal_pipe

Guido van Rossum added the comment: Rejected, in favor of #1583 which lets the app choose whether to use a fd or not. We had an extensive #python-dev discussion on this and this time the rejection is irrevocable. -- resolution: -> accepted status: open -> closed __

[issue1564547] Py_signal_pipe

Gustavo J. A. M. Carneiro added the comment: > The minimal patch doesn't initialize dummy_char or dummy_c. It's harmless here, but please fix it. ;) The variable is called 'dummy' for a reason. The value written or read is irrevelant... _ Tracker <[EMAIL PR

[issue1564547] Py_signal_pipe

Adam Olsen added the comment: The minimal patch doesn't initialize dummy_char or dummy_c. It's harmless here, but please fix it. ;) sizeof(dummy_char) will always be 1 (C defines sizeof as multiples of char.) The convention seems to be hardcoding 1 instead. ___

[issue1564547] Py_signal_pipe

Gustavo J. A. M. Carneiro added the comment: Minimal patch that just adds the pipe but does not attempt to fix anything else. Added file: http://bugs.python.org/file8898/python-signals-minimal.diff _ Tracker <[EMAIL PROTECTED]>

[issue1564547] Py_signal_pipe

Guido van Rossum added the comment: I've been told to look at this one more time. -- assignee: -> gvanrossum nosy: +gvanrossum resolution: rejected -> status: closed -> open _ Tracker <[EMAIL PROTECTED]> ___

[issue1564547] Py_signal_pipe

Gustavo J. A. M. Carneiro added the comment: "My understanding is that there are some things that the current patch does not address." Well, I don't know what those things are, so it's hard for me to address them. :-) _ Tracker <[EMAIL PROTECTED]>

[issue1564547] Py_signal_pipe

Sean Reifschneider added the comment: The thread at: http://bugzilla.gnome.org/show_bug.cgi?id=481569 requests "attention" so I'm formally rejecting it. :-) My reading of the python-dev thread on this is that the current patch does not reasonably fix the problem. However, I'd like mwh and an