Currently PulseView doesn't respond to Ctrl+C. Figure out how to make it do so.
"R. Diez" <rdiezmail-comparevcd@yahoo.de> writes.... Hi Joel: Please find attached a patch with the kind of small changes I would suggest for PulseView. How come PulseView ends up including Python headers through sigrokdecode.h? I was hoping that, if the GUI is just using the Python decoders, they would be encapsulated so that the GUI wouldn't need to know anything about Python itself. By the way, here is an example of how to cleanly exit on Ctrl+C (SIGINT). It uses sockets: http://bazaar.launchpad.net/~adamreichold/qpdfview/trunk/view/head:/sources/main.cpp http://bazaar.launchpad.net/~adamreichold/qpdfview/trunk/view/head:/sources/signalhandler.h http://bazaar.launchpad.net/~adamreichold/qpdfview/trunk/view/head:/sources/signalhandler.cpp More info about that technique is here: http://doc.qt.digia.com/stable/unix-signals.html Regards, rdiez
Created attachment 27 [details] A patch to add support for SIGINT.
(In reply to comment #2) > Created attachment 27 [details] > A patch to add support for SIGINT. I see nothing in attachment 27 [details] relating to sigint. For the record, I have had a fix for this issue since the beginning of the year: https://gitorious.org/pulseview/pulseview/commits/sigint
The sigint functionality is added in 31e6731d976cb2b398c92763a9d3507437b8a826 and 9d544917dcaa047c5bf7353d89c110792bae3d84 using the code from mrnuke, thanks! I've also merged the changes contained in the patch from R. Diez in other commits.