diff --git a/src/app/application.cpp b/src/app/application.cpp
index a289c8e36..7052051ff 100644
--- a/src/app/application.cpp
+++ b/src/app/application.cpp
@@ -178,11 +178,11 @@ QString Application::fileLoggerPath() const
             QVariant(specialFolderLocation(SpecialFolder::Data) + LOG_FOLDER)).toString();
 }
 
-void Application::setFileLoggerPath(const QString &value)
+void Application::setFileLoggerPath(const QString &path)
 {
     if (m_fileLogger)
-        m_fileLogger->changePath(value);
-    settings()->storeValue(KEY_FILELOGGER_PATH, value);
+        m_fileLogger->changePath(path);
+    settings()->storeValue(KEY_FILELOGGER_PATH, path);
 }
 
 bool Application::isFileLoggerBackup() const
diff --git a/src/base/bittorrent/session.cpp b/src/base/bittorrent/session.cpp
index 9c08c15ae..1e8a665d7 100644
--- a/src/base/bittorrent/session.cpp
+++ b/src/base/bittorrent/session.cpp
@@ -180,7 +180,7 @@ namespace
         {
         }
 
-        T operator()(T val)
+        T operator()(T val) const
         {
             return val <= m_limit ? m_ret : val;
         }
diff --git a/src/base/bittorrent/session.h b/src/base/bittorrent/session.h
index dc7d6cfcd..34724a75a 100644
--- a/src/base/bittorrent/session.h
+++ b/src/base/bittorrent/session.h
@@ -316,7 +316,7 @@ namespace BitTorrent
         bool isTrackerFilteringEnabled() const;
         void setTrackerFilteringEnabled(bool enabled);
         QStringList bannedIPs() const;
-        void setBannedIPs(const QStringList &list);
+        void setBannedIPs(const QStringList &newList);
 
         void startUpTorrents();
         TorrentHandle *findTorrent(const InfoHash &hash) const;
diff --git a/src/base/bittorrent/torrenthandle.cpp b/src/base/bittorrent/torrenthandle.cpp
index 0a52254b3..a6436ec34 100644
--- a/src/base/bittorrent/torrenthandle.cpp
+++ b/src/base/bittorrent/torrenthandle.cpp
@@ -74,6 +74,7 @@ AddTorrentData::AddTorrentData()
     : resumed(false)
     , disableTempPath(false)
     , sequential(false)
+    , firstLastPiecePriority(false)
     , hasSeedStatus(false)
     , skipChecking(false)
     , hasRootFolder(true)
@@ -1655,7 +1656,7 @@ void TorrentHandle::handleMetadataReceivedAlert(libt::metadata_received_alert *p
         m_speedMonitor.reset();
         m_session->handleTorrentPaused(this);
     }
-    
+
     // If first/last piece priority was specified when adding this torrent, we can set it
     // now that we have metadata:
     if (m_needsToSetFirstLastPiecePriority) {
diff --git a/src/base/bittorrent/trackerentry.cpp b/src/base/bittorrent/trackerentry.cpp
index 14f98e738..824f8f1ba 100644
--- a/src/base/bittorrent/trackerentry.cpp
+++ b/src/base/bittorrent/trackerentry.cpp
@@ -84,7 +84,7 @@ TrackerEntry &TrackerEntry::operator=(const TrackerEntry &other)
     return *this;
 }
 
-bool TrackerEntry::operator==(const TrackerEntry &other)
+bool TrackerEntry::operator==(const TrackerEntry &other) const
 {
     return (QUrl(url()) == QUrl(other.url()));
 }
diff --git a/src/base/bittorrent/trackerentry.h b/src/base/bittorrent/trackerentry.h
index f9d2187c4..fcaf9af7d 100644
--- a/src/base/bittorrent/trackerentry.h
+++ b/src/base/bittorrent/trackerentry.h
@@ -60,7 +60,7 @@ namespace BitTorrent
 
         void setTier(int value);
         TrackerEntry &operator=(const TrackerEntry &other);
-        bool operator==(const TrackerEntry &other);
+        bool operator==(const TrackerEntry &other) const;
 
         libtorrent::announce_entry nativeEntry() const;
 
diff --git a/src/base/net/smtp.h b/src/base/net/smtp.h
index b6a37749f..6c1b9b9ee 100644
--- a/src/base/net/smtp.h
+++ b/src/base/net/smtp.h
@@ -62,7 +62,7 @@ namespace Net
         Smtp(QObject *parent = 0);
         ~Smtp();
 
-        void sendMail(const QString &m_from, const QString &to, const QString &subject, const QString &body);
+        void sendMail(const QString &from, const QString &to, const QString &subject, const QString &body);
 
     private slots:
         void readyRead();
diff --git a/src/base/rss/rss_autodownloadrule.cpp b/src/base/rss/rss_autodownloadrule.cpp
index e32bfa584..48c194621 100644
--- a/src/base/rss/rss_autodownloadrule.cpp
+++ b/src/base/rss/rss_autodownloadrule.cpp
@@ -430,9 +430,9 @@ QStringList AutoDownloadRule::feedURLs() const
     return m_dataPtr->feedURLs;
 }
 
-void AutoDownloadRule::setFeedURLs(const QStringList &rssFeeds)
+void AutoDownloadRule::setFeedURLs(const QStringList &urls)
 {
-    m_dataPtr->feedURLs = rssFeeds;
+    m_dataPtr->feedURLs = urls;
 }
 
 QString AutoDownloadRule::name() const
diff --git a/src/base/rss/rss_autodownloadrule.h b/src/base/rss/rss_autodownloadrule.h
index d02d67b89..6b79fe3a5 100644
--- a/src/base/rss/rss_autodownloadrule.h
+++ b/src/base/rss/rss_autodownloadrule.h
@@ -59,7 +59,7 @@ namespace RSS
         QString mustNotContain() const;
         void setMustNotContain(const QString &tokens);
         QStringList feedURLs() const;
-        void setFeedURLs(const QStringList &feedURLs);
+        void setFeedURLs(const QStringList &urls);
         int ignoreDays() const;
         void setIgnoreDays(int d);
         QDateTime lastMatch() const;
@@ -74,7 +74,7 @@ namespace RSS
         TriStateBool addPaused() const;
         void setAddPaused(const TriStateBool &addPaused);
         QString assignedCategory() const;
-        void setCategory(const QString &assignedCategory);
+        void setCategory(const QString &category);
 
         bool matches(const QString &articleTitle) const;
 
diff --git a/src/base/torrentfilter.cpp b/src/base/torrentfilter.cpp
index 09daf45dc..db9519063 100644
--- a/src/base/torrentfilter.cpp
+++ b/src/base/torrentfilter.cpp
@@ -49,14 +49,14 @@ TorrentFilter::TorrentFilter()
 {
 }
 
-TorrentFilter::TorrentFilter(Type type, QStringSet hashSet, QString category)
+TorrentFilter::TorrentFilter(const Type type, const QStringSet &hashSet, const QString &category)
     : m_type(type)
     , m_category(category)
     , m_hashSet(hashSet)
 {
 }
 
-TorrentFilter::TorrentFilter(QString filter, QStringSet hashSet, QString category)
+TorrentFilter::TorrentFilter(const QString &filter, const QStringSet &hashSet, const QString &category)
     : m_type(All)
     , m_category(category)
     , m_hashSet(hashSet)
diff --git a/src/base/torrentfilter.h b/src/base/torrentfilter.h
index 1efde98be..06c18b346 100644
--- a/src/base/torrentfilter.h
+++ b/src/base/torrentfilter.h
@@ -72,8 +72,8 @@ public:
 
     TorrentFilter();
     // category: pass empty string for "no category" or null string (QString()) for "any category"
-    TorrentFilter(Type type, QStringSet hashSet = AnyHash, QString category = AnyCategory);
-    TorrentFilter(QString filter, QStringSet hashSet = AnyHash, QString category = AnyCategory);
+    TorrentFilter(const Type type, const QStringSet &hashSet = AnyHash, const QString &category = AnyCategory);
+    TorrentFilter(const QString &filter, const QStringSet &hashSet = AnyHash, const QString &category = AnyCategory);
 
     bool setType(Type type);
     bool setTypeByName(const QString &filter);
diff --git a/src/gui/addnewtorrentdialog.cpp b/src/gui/addnewtorrentdialog.cpp
index e0dc803e7..5bbac7bb4 100644
--- a/src/gui/addnewtorrentdialog.cpp
+++ b/src/gui/addnewtorrentdialog.cpp
@@ -78,8 +78,8 @@ namespace
 AddNewTorrentDialog::AddNewTorrentDialog(const BitTorrent::AddTorrentParams &inParams, QWidget *parent)
     : QDialog(parent)
     , ui(new Ui::AddNewTorrentDialog)
-    , m_contentModel(0)
-    , m_contentDelegate(0)
+    , m_contentModel(nullptr)
+    , m_contentDelegate(nullptr)
     , m_hasMetadata(false)
     , m_oldIndex(0)
     , m_torrentParams(inParams)
@@ -138,7 +138,7 @@ AddNewTorrentDialog::AddNewTorrentDialog(const BitTorrent::AddTorrentParams &inP
     // Signal / slots
     connect(ui->adv_button, SIGNAL(clicked(bool)), SLOT(showAdvancedSettings(bool)));
     connect(ui->doNotDeleteTorrentCheckBox, SIGNAL(clicked(bool)), SLOT(doNotDeleteTorrentClicked(bool)));
-    editHotkey = new QShortcut(Qt::Key_F2, ui->contentTreeView, 0, 0, Qt::WidgetShortcut);
+    QShortcut *editHotkey = new QShortcut(Qt::Key_F2, ui->contentTreeView, 0, 0, Qt::WidgetShortcut);
     connect(editHotkey, SIGNAL(activated()), SLOT(renameSelectedFile()));
     connect(ui->contentTreeView, SIGNAL(doubleClicked(QModelIndex)), SLOT(renameSelectedFile()));
 
@@ -148,10 +148,9 @@ AddNewTorrentDialog::AddNewTorrentDialog(const BitTorrent::AddTorrentParams &inP
 AddNewTorrentDialog::~AddNewTorrentDialog()
 {
     saveState();
+
+    delete m_contentDelegate;
     delete ui;
-    if (m_contentModel)
-        delete m_contentModel;
-    delete editHotkey;
 }
 
 bool AddNewTorrentDialog::isEnabled()
diff --git a/src/gui/addnewtorrentdialog.h b/src/gui/addnewtorrentdialog.h
index 7d9a153d2..ad28474d8 100644
--- a/src/gui/addnewtorrentdialog.h
+++ b/src/gui/addnewtorrentdialog.h
@@ -110,7 +110,6 @@ private:
     QString m_filePath;
     BitTorrent::InfoHash m_hash;
     BitTorrent::TorrentInfo m_torrentInfo;
-    QShortcut *editHotkey;
     QByteArray m_headerState;
     int m_oldIndex;
     QScopedPointer<TorrentFileGuard> m_torrentGuard;
diff --git a/src/gui/ico.h b/src/gui/ico.h
index a9cf548b8..414afb7ef 100644
--- a/src/gui/ico.h
+++ b/src/gui/ico.h
@@ -41,7 +41,7 @@ public:
     ICOHandler();
 
     bool canRead() const;
-    bool read(QImage *image);
+    bool read(QImage *outImage);
     bool write(const QImage &image);
 
     QByteArray name() const;
diff --git a/src/gui/mainwindow.h b/src/gui/mainwindow.h
index c94f8be7d..80966c20b 100644
--- a/src/gui/mainwindow.h
+++ b/src/gui/mainwindow.h
@@ -199,7 +199,7 @@ private:
     void dragEnterEvent(QDragEnterEvent *event) override;
     void closeEvent(QCloseEvent *) override;
     void showEvent(QShowEvent *) override;
-    bool event(QEvent *event) override;
+    bool event(QEvent *e) override;
     void displayRSSTab(bool enable);
     void displaySearchTab(bool enable);
 
diff --git a/src/gui/optionsdlg.h b/src/gui/optionsdlg.h
index ee874ba20..294a3c90c 100644
--- a/src/gui/optionsdlg.h
+++ b/src/gui/optionsdlg.h
@@ -82,7 +82,7 @@ public slots:
     void showConnectionTab();
 
 private slots:
-    void enableProxy(int comboIndex);
+    void enableProxy(int index);
     void on_buttonBox_accepted();
     void closeEvent(QCloseEvent *e);
     void on_buttonBox_rejected();
@@ -107,7 +107,7 @@ private slots:
     void on_btnWebUiCrt_clicked();
     void on_btnWebUiKey_clicked();
     void on_registerDNSBtn_clicked();
-    void setLocale(const QString &locale);
+    void setLocale(const QString &localeStr);
 
 private:
     // Methods
diff --git a/src/gui/search/searchsortmodel.cpp b/src/gui/search/searchsortmodel.cpp
index 4ef87af69..f5d24ff45 100644
--- a/src/gui/search/searchsortmodel.cpp
+++ b/src/gui/search/searchsortmodel.cpp
@@ -40,9 +40,9 @@ SearchSortModel::SearchSortModel(QObject *parent)
 {
 }
 
-void SearchSortModel::enableNameFilter(bool enable)
+void SearchSortModel::enableNameFilter(bool enabled)
 {
-    m_isNameFilterEnabled = enable;
+    m_isNameFilterEnabled = enabled;
 }
 
 void SearchSortModel::setNameFilter(const QString &searchTerm)
diff --git a/src/gui/transferlistfilterswidget.h b/src/gui/transferlistfilterswidget.h
index b2b7ad0f5..2a661a7a1 100644
--- a/src/gui/transferlistfilterswidget.h
+++ b/src/gui/transferlistfilterswidget.h
@@ -111,7 +111,7 @@ public slots:
 
 private slots:
     void handleFavicoDownload(const QString &url, const QString &filePath);
-    void handleFavicoFailure(const QString &url, const QString &reason);
+    void handleFavicoFailure(const QString &url, const QString &error);
 
 private:
     // These 4 methods are virtual slots in the base class.