Hi rdiez,
Thanks for your patch. I'm on a temporary PulseView hiatus, but I'll try and get
it merged as soon as I can - within the next few days. For now I've attached it
to a bug: http://sigrok.org/bugzilla/show_bug.cgi?id=66 so there's no chance it
can be forgotten.
Joel
On 12 February 2013 at 15:02 "R. Diez" <[email protected]> wrote:
> 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
------------------------------------------------------------------------------
Free Next-Gen Firewall Hardware Offer
Buy your Sophos next-gen firewall before the end March 2013
and get the hardware for free! Learn more.
http://p.sf.net/sfu/sophos-d2d-feb
_______________________________________________
sigrok-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sigrok-devel
