[Bug 47054] Make mission-control quit gracefully
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Tue Mar 13 15:30:11 CET 2012
https://bugs.freedesktop.org/show_bug.cgi?id=47054
--- Comment #1 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2012-03-13 07:30:11 PDT ---
Looks fine to me but here's some bikeshedding:
+static void
+sigusr1_cb (int sig)
+{
+ mcd_service_stop (mcd);
}
This doesn't seem to be the same indentation as the rest of the file?
+ g_object_unref (mcd);
use g_clear_object here if you want.
Simon is claiming that "using POSIX signals like that isn't safe
unfortunately". Over to you, Simon.
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the telepathy-bugs
mailing list