Comment 7 for bug 956416

Revision history for this message
xapantu (xapantu) wrote : Re: [Bug 956416] Re: segmentation fault on SIGINT

Thanks for your fix, I've just applied it to the trunk :)

  status fixcommitted
  importance high

Le 16/03/2012 16:01, Pim Vullers a écrit :
> The relevant piece of code, as far as I can see is this:
>
> protected static void sig_handler (int sig) {
> warning ("Caught signal (%d), exiting", sig);
> Granite.app.quit_mainloop ();
> }
>
> The warning still gets printed, but then it goes wrong.
> - For one it uses the deprecated Granite.app static variable.
> - quit_mainloop is a virtual function according to valadoc, so I guess this is the nullpointer which gets dereferenced
>
> It seems that the correct way to end would be to use Gtk.main_quit(),
> see http://www.valadoc.org/gtk+-3.0/Gtk.main_quit.html
>