diff --git a/src/gui/FavoritesManager.cxx b/src/gui/FavoritesManager.cxx
index a97b40b11..2c30787f8 100644
--- a/src/gui/FavoritesManager.cxx
+++ b/src/gui/FavoritesManager.cxx
@@ -220,7 +220,7 @@ void FavoritesManager::incPopular(const string& path)
   static constexpr uInt32 scale = 100;
   static constexpr double factor = 0.7;
   static constexpr uInt32 max_popular = scale;
-  static constexpr uInt32 min_popular = max_popular * factor;
+  //static constexpr uInt32 min_popular = max_popular * factor;
 
   auto increased = myPopularMap.find(path);
   if(increased != myPopularMap.end())
@@ -276,4 +276,4 @@ const FavoritesManager::PopularList& FavoritesManager::sortedPopularList(bool so
     return BSPF::compareIgnoreCase(aNode.getName(), bNode.getName()) < 0;
   });
   return sortedList;
-}
\ No newline at end of file
+}
diff --git a/src/gui/FavoritesManager.hxx b/src/gui/FavoritesManager.hxx
index 2658e032e..658adaaf4 100644
--- a/src/gui/FavoritesManager.hxx
+++ b/src/gui/FavoritesManager.hxx
@@ -39,7 +39,7 @@ class FavoritesManager
     using PopularType = std::pair<string, uInt32>;
     using PopularList = std::vector<PopularType>;
 
-    FavoritesManager::FavoritesManager(Settings& settings);
+    explicit FavoritesManager(Settings& settings);
 
     void load();
     void save();
@@ -87,4 +87,4 @@ class FavoritesManager
     FavoritesManager& operator=(FavoritesManager&&) = delete;
 };
 
-#endif
\ No newline at end of file
+#endif
diff --git a/src/gui/LauncherFileListWidget.cxx b/src/gui/LauncherFileListWidget.cxx
index 021862cf9..2e6441b63 100644
--- a/src/gui/LauncherFileListWidget.cxx
+++ b/src/gui/LauncherFileListWidget.cxx
@@ -184,7 +184,7 @@ void LauncherFileListWidget::removeFavorite()
 // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 void LauncherFileListWidget::userFavor(const string& path, bool isUserFavorite)
 {
-  int pos = 0;
+  size_t pos = 0;
 
   for(const auto& file : _fileList)
   {