Minor code clean up

src/mainwindow.cpp

@@ -107,7 +107,7 @@ MainWindow::MainWindow(QWidget *parent, const QStringList& torrentCmdLine) : QMa
   // Setting icons
 #if defined(Q_WS_X11)
   if (Preferences().useSystemIconTheme())
-    setWindowIcon(QIcon::fromTheme("qbittorrent", QString::fromUtf8(":/Icons/skin/qbittorrent32.png")));
+    setWindowIcon(QIcon::fromTheme("qbittorrent", QIcon(QString::fromUtf8(":/Icons/skin/qbittorrent32.png"))));
   else
 #else
     setWindowIcon(QIcon(QString::fromUtf8(":/Icons/skin/qbittorrent32.png")));
@@ -712,7 +712,7 @@ void MainWindow::toggleVisibility(QSystemTrayIcon::ActivationReason e) {
           return;
       }
       // Make sure the window is not minimized
-      setWindowState(windowState() & ~Qt::WindowMinimized | Qt::WindowActive);
+      setWindowState(windowState() & (~Qt::WindowMinimized | Qt::WindowActive));
       // Then show it
       show();
       raise();

src/qtlibtorrent/torrentmodel.cpp

@@ -37,14 +37,15 @@
 using namespace libtorrent;
 
 TorrentModelItem::TorrentModelItem(const QTorrentHandle &h)
+  : m_torrent(h)
+  , m_addedTime(TorrentPersistentData::getAddedDate(h.hash()))
+  , m_seedTime(TorrentPersistentData::getSeedDate(h.hash()))
+  , m_label(TorrentPersistentData::getLabel(h.hash()))
+  , m_name(TorrentPersistentData::getName(h.hash()))
+  , m_hash(h.hash())
 {
-  m_torrent = h;
+  if (m_name.isEmpty())
-  m_hash = h.hash();
+    m_name = h.name();
-  m_name = TorrentPersistentData::getName(h.hash());
-  if (m_name.isEmpty()) m_name = h.name();
-  m_addedTime = TorrentPersistentData::getAddedDate(h.hash());
-  m_seedTime = TorrentPersistentData::getSeedDate(h.hash());
-  m_label = TorrentPersistentData::getLabel(h.hash());
 }
 
 TorrentModelItem::State TorrentModelItem::state() const

src/qtlibtorrent/trackerinfos.h

@@ -40,13 +40,19 @@ public:
   unsigned long num_peers;
 
   //TrackerInfos() {}
-  TrackerInfos(const TrackerInfos &b) {
+  TrackerInfos(const TrackerInfos &b)
-    name_or_url = b.name_or_url;
+    : name_or_url(b.name_or_url)
+    , last_message(b.last_message)
+    , num_peers(b.num_peers)
+  {
     Q_ASSERT(!name_or_url.isEmpty());
-    last_message = b.last_message;
-    num_peers = b.num_peers;
   }
-  TrackerInfos(QString name_or_url): name_or_url(name_or_url), last_message(""), num_peers(0) {
+
+  TrackerInfos(QString name_or_url)
+    : name_or_url(name_or_url)
+    , last_message("")
+    , num_peers(0)
+  {
   }
 };
 

src/qtsingleapp/qtlockedfile.cpp

@@ -88,12 +88,12 @@
 */
 QtLockedFile::QtLockedFile()
     : QFile()
+    , m_lock_mode(NoLock)
 {
 #ifdef Q_OS_WIN
     wmutex = 0;
     rmutex = 0;
 #endif
-    m_lock_mode = NoLock;
 }
 
 /*!
@@ -105,12 +105,12 @@ QtLockedFile::QtLockedFile()
 */
 QtLockedFile::QtLockedFile(const QString &name)
     : QFile(name)
+    , m_lock_mode(NoLock)
 {
 #ifdef Q_OS_WIN
     wmutex = 0;
     rmutex = 0;
 #endif
-    m_lock_mode = NoLock;
 }
 
 /*!

src/webui/httpresponsegenerator.h

@@ -37,7 +37,7 @@
 class HttpResponseGenerator : public QHttpResponseHeader
 {
 
-	public:
+public:
     void setMessage(const QByteArray& message);
     void setMessage(const QString& message);
     void setContentTypeByExt(const QString& ext);