[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