Made the game counter more logical.

es-app/src/FileData.cpp

@@ -517,9 +517,8 @@ void FileData::sort(ComparisonFunction& comparator, bool ascending,
     for (auto it = mChildren.cbegin(); it != mChildren.cend(); it++) {
         // Game count, which will be displayed in the system view.
         if ((*it)->getType() == GAME && (*it)->getCountAsGame()) {
-            if (!(*it)->getFavorite())
+            gameCount.first++;
-                gameCount.first++;
+            if ((*it)->getFavorite())
-            else
                 gameCount.second++;
         }
 
@@ -581,9 +580,8 @@ void FileData::sortFavoritesOnTop(ComparisonFunction& comparator, bool ascending
 
         // Game count, which will be displayed in the system view.
         if (mChildren[i]->getType() == GAME && mChildren[i]->getCountAsGame()) {
-            if (!mChildren[i]->getFavorite())
+            gameCount.first++;
-                gameCount.first++;
+            if (mChildren[i]->getFavorite())
-            else
                 gameCount.second++;
         }
 
@@ -702,9 +700,8 @@ void FileData::countGames(std::pair<unsigned int, unsigned int>& gameCount)
 {
     for (unsigned int i = 0; i < mChildren.size(); i++) {
         if (mChildren[i]->getType() == GAME && mChildren[i]->getCountAsGame()) {
-            if (!mChildren[i]->getFavorite())
+            gameCount.first++;
-                gameCount.first++;
+            if (mChildren[i]->getFavorite())
-            else
                 gameCount.second++;
         }
         // Iterate through any folders.

es-app/src/FileData.h

@@ -146,7 +146,7 @@ private:
     std::unordered_map<std::string,FileData*> mChildrenByFilename;
     std::vector<FileData*> mChildren;
     std::vector<FileData*> mFilteredChildren;
-    // The pair includes non-favorite games, and favorite games.
+    // The pair includes all games, and favorite games.
     std::pair<unsigned int, unsigned int> mGameCount;
     bool mOnlyFolders;
     bool mHasFolders;

es-app/src/views/SystemView.cpp

@@ -284,19 +284,18 @@ void SystemView::onCursorChanged(const CursorState& /*state*/)
     // Also change the text after we've fully faded out.
     setAnimation(infoFadeOut, 0, [this, gameCount] {
         std::stringstream ss;
-        unsigned int totalGameCount = gameCount.first + gameCount.second;
 
         if (!getSelected()->isGameSystem())
             ss << "CONFIGURATION";
         else if (getSelected()->isCollection() && (getSelected()->getName() == "favorites"))
-            ss << totalGameCount << " GAME" << (totalGameCount == 1 ? " " : "S");
+            ss << gameCount.first << " GAME" << (gameCount.first == 1 ? " " : "S");
         // The 'recent' gamelist has probably been trimmed after sorting, so we'll cap it at
         // its maximum limit of 50 games.
         else if (getSelected()->isCollection() && (getSelected()->getName() == "recent"))
-            ss << (totalGameCount > 50 ? 50 : totalGameCount) << " GAME" <<
+            ss << (gameCount.first > 50 ? 50 : gameCount.first) << " GAME" <<
-                    (totalGameCount == 1 ? " " : "S");
+                    (gameCount.first == 1 ? " " : "S");
         else
-            ss << totalGameCount << " GAME" << (totalGameCount == 1 ? " " : "S ") << "(" <<
+            ss << gameCount.first << " GAME" << (gameCount.first == 1 ? " " : "S ") << "(" <<
                     gameCount.second << " FAVORITE" << (gameCount.second == 1 ? ")" : "S)");
 
         mSystemInfo.setText(ss.str());

es-app/src/views/gamelist/ISimpleGameListView.cpp

@@ -365,7 +365,7 @@ void ISimpleGameListView::generateGamelistInfo(FileData* cursor, FileData* first
 
     gameCount = rootFolder->getGameCount();
 
-    mGameCount = gameCount.first + gameCount.second;
+    mGameCount = gameCount.first;
     mFavoritesGameCount = gameCount.second;
     mFilteredGameCount = 0;
     mFilteredGameCountAll = 0;