Merge lp:~kamstrup/unity-lens-applications/finished-signal into lp:unity-lens-applications

Proposed by Mikkel Kamstrup Erlandsen
Status: Merged
Merged at revision: 184
Proposed branch: lp:~kamstrup/unity-lens-applications/finished-signal
Merge into: lp:unity-lens-applications
Diff against target: 21 lines (+4/-0)
1 file modified
src/daemon.vala (+4/-0)
To merge this branch: bzr merge lp:~kamstrup/unity-lens-applications/finished-signal
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+54187@code.launchpad.net

Description of the change

Use the new PlaceSearch.finished() API. This requires lp:~kamstrup/libunity/finished-signal which evidently can not be expressed in LP..?

To post a comment you must log in.
Revision history for this message
Neil J. Patel (njpatel) wrote :

Approved and, yeah, it can't be expressed :(

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/daemon.vala'
2--- src/daemon.vala 2011-03-14 13:26:08 +0000
3+++ src/daemon.vala 2011-03-21 11:34:23 +0000
4@@ -488,6 +488,8 @@
5 applications.thaw_notify ();
6 return false;
7 });
8+
9+ search.finished ();
10 }
11
12 private async void update_global_search (PlaceSearch? search)
13@@ -544,6 +546,8 @@
14 applications.thaw_notify ();
15 return false;
16 });
17+
18+ search.finished ();
19 }
20
21 private string prepare_pkg_search_string (PlaceSearch? search, Section section)

Subscribers

People subscribed via source and target branches