From fb55ea21c891a75f2e7e06d336faa9b5b8b105a3 Mon Sep 17 00:00:00 2001
From: Javier <dev.git@javispedro.com>
Date: Sun, 14 Jun 2015 20:18:59 +0200
Subject: improve sync locking and status messages

---
 smartpenmanager.cc | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

(limited to 'smartpenmanager.cc')

diff --git a/smartpenmanager.cc b/smartpenmanager.cc
index 1750589..0ac56e2 100644
--- a/smartpenmanager.cc
+++ b/smartpenmanager.cc
@@ -65,7 +65,7 @@ QStringList SmartpenManager::pensBeingSynchronized() const
 		QString name = it.value()->penName();
 		if (name.isEmpty()) {
 			Smartpen::Address addr = it.value()->penAddress();
-			name = QString("%1-%2").arg(addr.first, addr.second);
+			name = QString("%1-%2").arg(addr.first).arg(addr.second);
 		}
 		pens.append(name);
 	}
@@ -89,10 +89,20 @@ void SmartpenManager::handleSyncerFinished()
 {
 	SmartpenSyncer *syncer = static_cast<SmartpenSyncer*>(sender());
 	Smartpen::Address addr = syncer->penAddress();
+
 	qDebug() << "Finished synchronization with pen with address:" << addr;
+
 	_syncers.remove(addr);
-	syncer->deleteLater();
 	emit pensBeingSynchronizedChanged();
+
+	if (syncer->hasErrors()) {
+		qWarning() << "Synchronization with address" << addr << "failed";
+		emit syncFailed(syncer->penName());
+	} else {
+		emit syncComplete(syncer->penName());
+	}
+
+	syncer->deleteLater();
 }
 
 void SmartpenManager::processDevice(udev_device *dev)
-- 
cgit v1.2.3