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

Proposed by Mikkel Kamstrup Erlandsen
Status: Merged
Merged at revision: 162
Proposed branch: lp:~kamstrup/unity-lens-files/finished-signal
Merge into: lp:unity-lens-files
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-files/finished-signal
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+54188@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

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/daemon.vala'
--- src/daemon.vala 2011-03-16 11:58:08 +0000
+++ src/daemon.vala 2011-03-21 11:34:28 +0000
@@ -642,6 +642,8 @@
642 files.thaw_notify ();642 files.thaw_notify ();
643 return false;643 return false;
644 });644 });
645
646 search.finished ();
645 }647 }
646 648
647 private async void update_search_async (PlaceSearch search,649 private async void update_search_async (PlaceSearch search,
@@ -777,6 +779,8 @@
777 files.thaw_notify ();779 files.thaw_notify ();
778 return false;780 return false;
779 });781 });
782
783 search.finished ();
780 }784 }
781785
782 private async void update_without_search_async (Section section,786 private async void update_without_search_async (Section section,

Subscribers

People subscribed via source and target branches

to all changes: