From a9ce197d04a961a39d6dcd0d9b2b0037f6c2b72f Mon Sep 17 00:00:00 2001
From: Christophe Dumez <chris@qbittorrent.org>
Date: Thu, 24 Feb 2011 17:22:04 +0000
Subject: [PATCH] Improvement to last commit

---
 src/properties/trackerlist.cpp       | 1 +
 src/properties/trackersadditiondlg.h | 9 ++++++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/properties/trackerlist.cpp b/src/properties/trackerlist.cpp
index cf240348f..afbf27064 100644
--- a/src/properties/trackerlist.cpp
+++ b/src/properties/trackerlist.cpp
@@ -295,6 +295,7 @@ void TrackerList::askForTrackers(){
   QStringList trackers = TrackersAdditionDlg::askForTrackers(h);
   if(!trackers.empty()) {
     foreach(const QString& tracker, trackers) {
+      if(tracker.trimmed().isEmpty()) continue;
       announce_entry url(tracker.toStdString());
       url.tier = 0;
       h.add_tracker(url);
diff --git a/src/properties/trackersadditiondlg.h b/src/properties/trackersadditiondlg.h
index c36f5bc28..536f64b16 100644
--- a/src/properties/trackersadditiondlg.h
+++ b/src/properties/trackersadditiondlg.h
@@ -66,6 +66,7 @@ public:
 
 public slots:
   void on_uTorrentListButton_clicked() {
+    uTorrentListButton->setEnabled(false);
     DownloadThread *d = new DownloadThread(this);
     connect(d, SIGNAL(downloadFinished(QString,QString)), SLOT(parseUTorrentList(QString,QString)));
     connect(d, SIGNAL(downloadFailure(QString,QString)), SLOT(getTrackerError(QString,QString)));
@@ -79,6 +80,8 @@ public slots:
     if (!list_file.open(QFile::ReadOnly)) {
       QMessageBox::warning(this, tr("I/O Error"), tr("Error while trying to open the downloaded file."), QMessageBox::Ok);
       setCursor(Qt::ArrowCursor);
+      uTorrentListButton->setEnabled(true);
+      sender()->deleteLater();
       return;
     }
     QList<QUrl> existingTrackers;
@@ -114,16 +117,20 @@ public slots:
     list_file.remove();
     //To restore the cursor ...
     setCursor(Qt::ArrowCursor);
+    uTorrentListButton->setEnabled(true);
     // Display information message if necessary
     if(nb == 0) {
       QMessageBox::information(this, tr("No change"), tr("No additional trackers were found."), QMessageBox::Ok);
     }
+    sender()->deleteLater();
   }
 
-  void getTrackerError(QString, QString error) {
+  void getTrackerError(const QString&, const QString &error) {
     //To restore the cursor ...
     setCursor(Qt::ArrowCursor);
+    uTorrentListButton->setEnabled(true);
     QMessageBox::warning(this, tr("Download error"), tr("The trackers list could not be downloaded, reason: %1").arg(error), QMessageBox::Ok);
+    sender()->deleteLater();
   }
 
 public: