Unity

Code review comment for lp:~unity-team/unity/more-places

  1. more-places
  2. Merge into trunk
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Approved!

It seems to be a bit heavy on the IO since you create all those GDesktopAppInfo's (which parses the keyfile each time I believe) but it works fine. I think we should address that later with some caching magic.

review: Approve

« Back to merge proposal