Call TorrentModelItem::state() directly in getTorrentStatusReport

There is no point in wraping/unwraping QVariant.

src/qtlibtorrent/torrentmodel.cpp

@@ -504,7 +504,7 @@ TorrentStatusReport TorrentModel::getTorrentStatusReport() const
   QList<TorrentModelItem*>::const_iterator it = m_torrents.constBegin();
   QList<TorrentModelItem*>::const_iterator itend = m_torrents.constEnd();
   for ( ; it != itend; ++it) {
-    switch((*it)->data(TorrentModelItem::TR_STATUS).toInt()) {
+    switch((*it)->state()) {
     case TorrentModelItem::STATE_DOWNLOADING:
       ++report.nb_active;
       ++report.nb_downloading;

src/qtlibtorrent/torrentmodel.h

@@ -58,6 +58,7 @@ public:
   QVariant data(int column, int role = Qt::DisplayRole) const;
   bool setData(int column, const QVariant &value, int role = Qt::DisplayRole);
   inline QString hash() const { return m_hash; }
+  State state() const;
 
 signals:
   void labelChanged(QString previous, QString current);
@@ -65,7 +66,6 @@ signals:
 private:
   static QIcon getIconByState(State state);
   static QColor getColorByState(State state);
-  State state() const;
 
 private:
   QTorrentHandle m_torrent;