Code clean up

src/qtlibtorrent/qbtsession.cpp

@@ -802,12 +802,13 @@ void QBtSession::pauseAllTorrents() {
   std::vector<torrent_handle> torrents = s->get_torrents();
   std::vector<torrent_handle>::iterator torrentIT;
   for(torrentIT = torrents.begin(); torrentIT != torrents.end(); torrentIT++) {
-    QTorrentHandle h = QTorrentHandle(*torrentIT);
-    if(!h.is_valid()) continue;
-    if(!h.is_paused()) {
-      h.pause();
-      emit pausedTorrent(h);
-    }
+    try {
+      QTorrentHandle h = QTorrentHandle(*torrentIT);
+      if(!h.is_paused()) {
+        h.pause();
+        emit pausedTorrent(h);
+      }
+    } catch(invalid_handle&) {}
   }
 }
 
@@ -819,12 +820,13 @@ void QBtSession::resumeAllTorrents() {
   std::vector<torrent_handle> torrents = s->get_torrents();
   std::vector<torrent_handle>::iterator torrentIT;
   for(torrentIT = torrents.begin(); torrentIT != torrents.end(); torrentIT++) {
-    QTorrentHandle h = QTorrentHandle(*torrentIT);
-    if(!h.is_valid()) continue;
-    if(h.is_paused()) {
-      h.resume();
-      emit resumedTorrent(h);
-    }
+    try {
+      QTorrentHandle h = QTorrentHandle(*torrentIT);
+      if(h.is_paused()) {
+        h.resume();
+        emit resumedTorrent(h);
+      }
+    } catch(invalid_handle&) {}
   }
 }
 

src/transferlistwidget.cpp

@@ -131,10 +131,6 @@ TorrentModel* TransferListWidget::getSourceModel() const {
   return listModel;
 }
 
-int TransferListWidget::getNbTorrents() const {
-  return listModel->rowCount();
-}
-
 void TransferListWidget::previewFile(QString filePath) {
 #ifdef Q_WS_WIN
   QDesktopServices::openUrl(QUrl(QString("file:///")+filePath));
@@ -260,15 +256,6 @@ void TransferListWidget::startSelectedTorrents() {
   }
 }
 
-void TransferListWidget::startAllTorrents() {
-  for(int i=0; i<listModel->rowCount(); ++i) {
-    QTorrentHandle h = BTSession->getTorrentHandle(getHashFromRow(i));
-    if(h.is_valid() && h.is_paused()) {
-      h.resume();
-    }
-  }
-}
-
 void TransferListWidget::startVisibleTorrents() {
   QStringList hashes;
   for(int i=0; i<nameFilterModel->rowCount(); ++i) {
@@ -293,15 +280,6 @@ void TransferListWidget::pauseSelectedTorrents() {
   }
 }
 
-void TransferListWidget::pauseAllTorrents() {
-  for(int i=0; i<listModel->rowCount(); ++i) {
-    QTorrentHandle h = BTSession->getTorrentHandle(getHashFromRow(i));
-    if(h.is_valid() && !h.is_paused()) {
-      h.pause();
-    }
-  }
-}
-
 void TransferListWidget::pauseVisibleTorrents() {
   QStringList hashes;
   for(int i=0; i<nameFilterModel->rowCount(); ++i) {

src/transferlistwidget.h

@@ -50,7 +50,6 @@ class TransferListWidget: public QTreeView {
 public:
   TransferListWidget(QWidget *parent, MainWindow *main_window, QBtSession* BTSession);
   ~TransferListWidget();
-  int getNbTorrents() const;
   TorrentModel* getSourceModel() const;
 
 public slots:
@@ -58,10 +57,8 @@ public slots:
   void setRefreshInterval(int t);
   void setSelectedTorrentsLocation();
   void startSelectedTorrents();
-  void startAllTorrents();
   void startVisibleTorrents();
   void pauseSelectedTorrents();
-  void pauseAllTorrents();
   void pauseVisibleTorrents();
   void deleteSelectedTorrents();
   void deleteVisibleTorrents();