From 137df18a00bb419bc51d894a4d4e28ad6eaff69f Mon Sep 17 00:00:00 2001
From: kaienfr <kaienfr@gmail.com>
Date: Thu, 5 Sep 2013 00:04:24 +0200
Subject: [PATCH 1/2] Update recent games list & remove redundant recent games
 from ini file

---
 Core/Config.cpp           | 24 ++++++++++++++++++------
 UI/GameSettingsScreen.cpp |  7 +++++++
 UI/GameSettingsScreen.h   |  3 ++-
 UI/MainScreen.cpp         |  3 +++
 Windows/WndMainWindow.cpp |  5 +++++
 Windows/WndMainWindow.h   |  1 +
 6 files changed, 36 insertions(+), 7 deletions(-)

diff --git a/Core/Config.cpp b/Core/Config.cpp
index bb958935c0..3bb174408b 100644
--- a/Core/Config.cpp
+++ b/Core/Config.cpp
@@ -273,12 +273,13 @@ void Config::Save() {
 		IniFile::Section *recent = iniFile.GetOrCreateSection("Recent");
 		recent->Set("MaxRecent", iMaxRecent);
 	
-		for (int i = 0; i < (int)recentIsos.size(); i++)
-		{
+		for (int i = 0; i <  iMaxRecent; i++) {
 			char keyName[64];
-			
 			sprintf(keyName,"FileName%d",i);
-			recent->Set(keyName,recentIsos[i]);
+            if (i < recentIsos.size()){
+				recent->Set(keyName, recentIsos[i]);}
+			else{
+				recent->Delete(keyName);} // delete the nonexisting FileName
 		}
 
 		IniFile::Section *cpu = iniFile.GetOrCreateSection("CPU");
@@ -398,8 +399,19 @@ void Config::AddRecent(const std::string &file) {
 void Config::CleanRecent() {
 	std::vector<std::string> cleanedRecent;
 	for (size_t i = 0; i < recentIsos.size(); i++) {
-		if (File::Exists(recentIsos[i]))
-			cleanedRecent.push_back(recentIsos[i]);
+		if (File::Exists(recentIsos[i])){
+			// clean the redundant recent games' list.
+			if (cleanedRecent.size()==0){ // add first one
+					cleanedRecent.push_back(recentIsos[i]);
+			}
+			for (size_t j=0; j<cleanedRecent.size();j++){
+				if ((std::string)cleanedRecent[j]==(std::string)recentIsos[i])
+					break; // skip if found redundant
+				if (j==cleanedRecent.size()-1){ // add if no redundant found
+					cleanedRecent.push_back(recentIsos[i]);
+				}
+			}
+		}
 	}
 	recentIsos = cleanedRecent;
 }
diff --git a/UI/GameSettingsScreen.cpp b/UI/GameSettingsScreen.cpp
index 8a00e05b47..2f361983d3 100644
--- a/UI/GameSettingsScreen.cpp
+++ b/UI/GameSettingsScreen.cpp
@@ -48,6 +48,7 @@ namespace MainWindow {
 		WM_USER_LOG_STATUS_CHANGED = WM_USER + 101,
 		WM_USER_ATRAC_STATUS_CHANGED = WM_USER + 102,
 		WM_USER_UPDATE_UI = WM_USER + 103,
+		WM_USER_RECREATE_RECENTLIST = WM_USER + 104,
 	};
 	extern HWND hwndMain;
 }
@@ -398,6 +399,7 @@ void GameSettingsScreen::CreateViews() {
 
 UI::EventReturn GameSettingsScreen::OnClearRecents(UI::EventParams &e) {
 	g_Config.recentIsos.clear();
+	clearrecentlist_=true; // set true to send clear_recent_list message
 
 	return UI::EVENT_DONE;
 }
@@ -489,6 +491,11 @@ UI::EventReturn GameSettingsScreen::OnBack(UI::EventParams &e) {
 
 	KeyMap::UpdateConfirmCancelKeys();
 
+	if (clearrecentlist_){
+		// clear the recent game list on MainWindow.
+		PostMessage(MainWindow::hwndMain, MainWindow::WM_USER_RECREATE_RECENTLIST, 0, 0);
+		clearrecentlist_ = false; }
+
 	return UI::EVENT_DONE;
 }
 
diff --git a/UI/GameSettingsScreen.h b/UI/GameSettingsScreen.h
index 505a7a15a8..05f6b81fd7 100644
--- a/UI/GameSettingsScreen.h
+++ b/UI/GameSettingsScreen.h
@@ -25,7 +25,7 @@
 class GameSettingsScreen : public UIDialogScreenWithBackground {
 public:
 	GameSettingsScreen(std::string gamePath, std::string gameID = "")
-		: gamePath_(gamePath), gameID_(gameID), iAlternateSpeedPercent_(3), enableReports_(false) {}
+		: gamePath_(gamePath), gameID_(gameID), iAlternateSpeedPercent_(3), enableReports_(false), clearrecentlist_(false) {}
 
 	virtual void update(InputState &input);
 
@@ -64,6 +64,7 @@ private:
 	bool cap60FPS_;
 	int iAlternateSpeedPercent_;
 	bool enableReports_;
+	bool clearrecentlist_;
 };
 
 /*
diff --git a/UI/MainScreen.cpp b/UI/MainScreen.cpp
index cba0c91244..b479c6a558 100644
--- a/UI/MainScreen.cpp
+++ b/UI/MainScreen.cpp
@@ -554,6 +554,9 @@ void MainScreen::sendMessage(const char *message, const char *value) {
 	if (!strcmp(message, "language")) {
 		RecreateViews();
 	}
+	if (!strcmp(message, "clearrecentlist")){
+		RecreateViews();
+	}
 }
 
 void MainScreen::update(InputState &input) {
diff --git a/Windows/WndMainWindow.cpp b/Windows/WndMainWindow.cpp
index 49c7b1efda..e942f2fe4f 100644
--- a/Windows/WndMainWindow.cpp
+++ b/Windows/WndMainWindow.cpp
@@ -1538,6 +1538,11 @@ namespace MainWindow
 			Update();
 			break;
 
+		case WM_USER_RECREATE_RECENTLIST:
+			NativeMessageReceived("clearrecentlist", "");
+			Update();
+			break;
+
 		case WM_MENUSELECT:
 			// Unfortunately, accelerate keys (hotkeys) shares the same enabled/disabled states
 			// with corresponding menu items.
diff --git a/Windows/WndMainWindow.h b/Windows/WndMainWindow.h
index 1723a77c20..229390d9d2 100644
--- a/Windows/WndMainWindow.h
+++ b/Windows/WndMainWindow.h
@@ -12,6 +12,7 @@ namespace MainWindow
 		WM_USER_LOG_STATUS_CHANGED = WM_USER + 101,
 		WM_USER_ATRAC_STATUS_CHANGED = WM_USER + 102,
 		WM_USER_UPDATE_UI = WM_USER + 103,
+		WM_USER_RECREATE_RECENTLIST = WM_USER + 104,
 	};
 
 	enum {

From 701e1263f020298f60026376284247db4a79eefb Mon Sep 17 00:00:00 2001
From: kaienfr <kaienfr@gmail.com>
Date: Thu, 5 Sep 2013 18:45:41 +0200
Subject: [PATCH 2/2] New update in code style & fix load function

The load function can not load all recent games list if the Key is skeeped as:
FileName0 = ...
FileName2 = ...
...

In this case, only FileName0 is loaded.
Now the function is fixed and load all list.
---
 Core/Config.cpp           | 20 +++++++++++++-------
 UI/GameSettingsScreen.cpp |  5 +++--
 2 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/Core/Config.cpp b/Core/Config.cpp
index 3bb174408b..d4edc21263 100644
--- a/Core/Config.cpp
+++ b/Core/Config.cpp
@@ -99,8 +99,12 @@ void Config::Load(const char *iniFileName, const char *controllerIniFilename)
 		std::string fileName;
 
 		sprintf(keyName,"FileName%d",i);
-		if (!recent->Get(keyName,&fileName,"") || fileName.length() == 0) break;
-		recentIsos.push_back(fileName);
+		if (!recent->Get(keyName,&fileName,"") || fileName.length() == 0) {
+			// just skip it to get the next key
+		}
+		else {
+			recentIsos.push_back(fileName);
+		}
 	}
 
 	IniFile::Section *cpu = iniFile.GetOrCreateSection("CPU");
@@ -276,10 +280,12 @@ void Config::Save() {
 		for (int i = 0; i <  iMaxRecent; i++) {
 			char keyName[64];
 			sprintf(keyName,"FileName%d",i);
-            if (i < recentIsos.size()){
-				recent->Set(keyName, recentIsos[i]);}
-			else{
-				recent->Delete(keyName);} // delete the nonexisting FileName
+			if (i < recentIsos.size()) {
+				recent->Set(keyName, recentIsos[i]);
+			}
+			else {
+				recent->Delete(keyName); // delete the nonexisting FileName
+			} 
 		}
 
 		IniFile::Section *cpu = iniFile.GetOrCreateSection("CPU");
@@ -405,7 +411,7 @@ void Config::CleanRecent() {
 					cleanedRecent.push_back(recentIsos[i]);
 			}
 			for (size_t j=0; j<cleanedRecent.size();j++){
-				if ((std::string)cleanedRecent[j]==(std::string)recentIsos[i])
+				if (cleanedRecent[j]==recentIsos[i])
 					break; // skip if found redundant
 				if (j==cleanedRecent.size()-1){ // add if no redundant found
 					cleanedRecent.push_back(recentIsos[i]);
diff --git a/UI/GameSettingsScreen.cpp b/UI/GameSettingsScreen.cpp
index 2f361983d3..a4dfe56563 100644
--- a/UI/GameSettingsScreen.cpp
+++ b/UI/GameSettingsScreen.cpp
@@ -1,4 +1,4 @@
-// Copyright (c) 2013- PPSSPP Project.
+// Copyright (c) 2013- PPSSPP Project.
 
 // This program is free software: you can redistribute it and/or modify
 // it under the terms of the GNU General Public License as published by
@@ -494,7 +494,8 @@ UI::EventReturn GameSettingsScreen::OnBack(UI::EventParams &e) {
 	if (clearrecentlist_){
 		// clear the recent game list on MainWindow.
 		PostMessage(MainWindow::hwndMain, MainWindow::WM_USER_RECREATE_RECENTLIST, 0, 0);
-		clearrecentlist_ = false; }
+		clearrecentlist_ = false; 
+	}
 
 	return UI::EVENT_DONE;
 }