The latest patch looks good to me, but with a couple of notes (no need
to fix these):

 - using the _full() version of the timeout call is not necessary here

 - now that this patch is simplified it becomes easy to see how all of
this is just putting values into one structure and then later moving
them into another structure.  It seems like we could have just used the
first structure type, or even just fixed the other queue mechanism to
add the delay.  Oh well.  No real gain from making that change at this
point.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to gnome-menus in Ubuntu.
https://bugs.launchpad.net/bugs/1506744

Title:
  Newly installed applications do not show in the dash

To manage notifications about this bug go to:
https://bugs.launchpad.net/glib/+bug/1506744/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs

Reply via email to