Code review comment for lp:~alan-griffiths/compiz-compizconfig-gconf/Bug-953214

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

g_main_loop_is_running(NULL) looks odd to me. The code for that function is:

gboolean
g_main_loop_is_running (GMainLoop *loop)
{
  g_return_val_if_fail (loop != NULL, FALSE);
  g_return_val_if_fail (g_atomic_int_get (&loop->ref_count) > 0, FALSE);

  return loop->is_running;
}

So this statement always returns FALSE. So you've effectively just removed the processEvents(0) call.

« Back to merge proposal