[Merge] lp:~renatofilho/sync-monitor/fix-1579936 into lp:sync-monitor

Michael Zanetti michael.zanetti at canonical.com
Mon May 9 23:01:23 UTC 2016


Review: Approve

I don't think the explicit Qt::DirectConnection is needed here. That should be the case nevertheless.

Anyhow, the moving around of the emits syncAboutToStart() before the actual sync() call definitely makes sense. Also I have tested this here and it fixes my wakelock situation indeed.
-- 
https://code.launchpad.net/~renatofilho/sync-monitor/fix-1579936/+merge/294169
Your team Ubuntu Phablet Team is subscribed to branch lp:sync-monitor.



More information about the Ubuntu-reviews mailing list