Comment 10 for bug 1427344

Revision history for this message
Tim Waugh (twaugh) wrote :

From a quick glance, this looks wrong:

```
  /* If auto shutdown is active we have perhaps scheduled a timer to shut down
     due to not having queues any more to maintain, kill the timer now */
  if (autoshutdown && autoshutdown_exec_id &&
      cupsArrayCount(remote_printers) > 0) {
    debug_printf ("cups-browsed: New printers there to make available, killing auto shutdown timer.\n");
    g_source_destroy(g_main_context_find_source_by_id(NULL,
                                                      autoshutdown_exec_id));
    autoshutdown_exec_id = 0;
  }
```

Why are you calling g_source_destroy() rather than g_source_remove()?