Savedata management screen

CMakeLists.txt

@@ -820,6 +820,7 @@ set(NativeAppSource
 	UI/OnScreenDisplay.cpp
 	UI/ControlMappingScreen.cpp
 	UI/ReportScreen.cpp
+	UI/SavedataScreen.cpp
 	UI/Store.cpp
 	UI/CwCheatScreen.cpp
 	UI/InstallZipScreen.cpp

Core/HLE/sceIo.cpp

@@ -317,8 +317,6 @@ static void __IoFreeFd(int fd, u32 &error) {
 // TODO: We don't do any of that yet.
 // For now, let's at least delay the callback notification.
 static void __IoAsyncNotify(u64 userdata, int cyclesLate) {
-	PROFILE_THIS_SCOPE("io_rw");
-
 	int fd = (int) userdata;
 
 	u32 error;

Core/Loaders.cpp

@@ -758,7 +758,7 @@ IdentifiedFileType Identify_File(FileLoader *fileLoader)
 	}
 
 	std::string extension = fileLoader->Extension();
-	if (!strcasecmp(extension.c_str(),".iso"))
+	if (!strcasecmp(extension.c_str(), ".iso"))
 	{
 		// may be a psx iso, they have 2352 byte sectors. You never know what some people try to open
 		if ((fileLoader->FileSize() % 2352) == 0)
@@ -780,26 +780,36 @@ IdentifiedFileType Identify_File(FileLoader *fileLoader)
 	{
 		return FILETYPE_PSP_ISO;
 	}
-
+	else if (!strcasecmp(extension.c_str(),".ppst"))
+	{
+		return FILETYPE_PPSSPP_SAVESTATE;
+	}
 
 	// First, check if it's a directory with an EBOOT.PBP in it.
 	if (fileLoader->IsDirectory()) {
 		std::string filename = fileLoader->Path();
 		if (filename.size() > 4) {
-			FileInfo ebootInfo;
+			FileInfo fileInfo;
 			// Check for existence of EBOOT.PBP, as required for "Directory games".
-			if (getFileInfo((filename + "/EBOOT.PBP").c_str(), &ebootInfo)) {
-				if (ebootInfo.exists) {
+			if (getFileInfo((filename + "/EBOOT.PBP").c_str(), &fileInfo)) {
+				if (fileInfo.exists) {
 					return FILETYPE_PSP_PBP_DIRECTORY;
 				}
 			}
 
 			// check if it's a disc directory
-			if (getFileInfo((filename + "/PSP_GAME").c_str(), &ebootInfo)) {
-				if (ebootInfo.exists) {
+			if (getFileInfo((filename + "/PSP_GAME").c_str(), &fileInfo)) {
+				if (fileInfo.exists) {
 					return FILETYPE_PSP_DISC_DIRECTORY;
 				}
 			}
+
+			// Not that, okay, let's guess it's a savedata directory if it has a param.sfo...
+			if (getFileInfo((filename + "/PARAM.SFO").c_str(), &fileInfo)) {
+				if (fileInfo.exists) {
+					return FILETYPE_PSP_SAVEDATA_DIRECTORY;
+				}
+			}
 		}
 
 		return FILETYPE_NORMAL_DIRECTORY;

Core/Loaders.h

@@ -43,6 +43,9 @@ enum IdentifiedFileType {
 
 	FILETYPE_NORMAL_DIRECTORY,
 
+	FILETYPE_PSP_SAVEDATA_DIRECTORY,
+	FILETYPE_PPSSPP_SAVESTATE,
+
 	FILETYPE_UNKNOWN
 };
 

Core/System.cpp

@@ -529,6 +529,8 @@ std::string GetSysDirectory(PSPDirectories directoryType) {
 		return g_Config.memStickDirectory + "PAUTH/";
 	case DIRECTORY_DUMP:
 		return g_Config.memStickDirectory + "PSP/SYSTEM/DUMP/";
+	case DIRECTORY_SAVESTATE:
+		return g_Config.memStickDirectory + "PSP/PPSSPP_STATE/";
 	// Just return the memory stick root if we run into some sort of problem.
 	default:
 		ERROR_LOG(FILESYS, "Unknown directory type.");

Core/System.h

@@ -43,6 +43,7 @@ enum PSPDirectories {
 	DIRECTORY_SAVEDATA,
 	DIRECTORY_PAUTH,
 	DIRECTORY_DUMP,
+	DIRECTORY_SAVESTATE,
 };
 
 

UI/DevScreens.h

@@ -31,9 +31,8 @@ class DevMenu : public PopupScreen {
 public:
 	DevMenu() : PopupScreen("Dev Tools") {}
 
-	virtual void CreatePopupContents(UI::ViewGroup *parent);
-
-	virtual void dialogFinished(const Screen *dialog, DialogResult result);
+	void CreatePopupContents(UI::ViewGroup *parent) override;
+	void dialogFinished(const Screen *dialog, DialogResult result) override;
 
 protected:
 	UI::EventReturn OnLogView(UI::EventParams &e);
@@ -48,7 +47,7 @@ class DevMenu : public PopupScreen {
 class LogConfigScreen : public UIDialogScreenWithBackground {
 public:
 	LogConfigScreen() {}
-	virtual void CreateViews();
+	virtual void CreateViews() override;
 
 private:
 	UI::EventReturn OnToggleAll(UI::EventParams &e);

UI/GameInfoCache.cpp

@@ -48,7 +48,7 @@ GameInfo::~GameInfo() {
 	delete fileLoader;
 }
 
-bool GameInfo::DeleteGame() {
+bool GameInfo::Delete() {
 	switch (fileType) {
 	case FILETYPE_PSP_ISO:
 	case FILETYPE_PSP_ISO_NP:
@@ -63,9 +63,9 @@ bool GameInfo::DeleteGame() {
 			return true;
 		}
 	case FILETYPE_PSP_PBP_DIRECTORY:
+	case FILETYPE_PSP_SAVEDATA_DIRECTORY:
 		{
 			// TODO: This could be handled by Core/Util/GameManager too somehow.
-
 			const char *directoryToRemove = filePath_.c_str();
 			INFO_LOG(HLE, "Deleting %s", directoryToRemove);
 			if (!File::DeleteDirRecursively(directoryToRemove)) {
@@ -76,6 +76,12 @@ bool GameInfo::DeleteGame() {
 			return true;
 		}
 	case FILETYPE_PSP_ELF:
+	case FILETYPE_PPSSPP_SAVESTATE:
+	case FILETYPE_UNKNOWN_BIN:
+	case FILETYPE_UNKNOWN_ELF:
+	case FILETYPE_ARCHIVE_RAR:
+	case FILETYPE_ARCHIVE_ZIP:
+	case FILETYPE_ARCHIVE_7Z:
 		{
 			const char *fileToRemove = filePath_.c_str();
 			deleteFile(fileToRemove);
@@ -90,13 +96,27 @@ bool GameInfo::DeleteGame() {
 u64 GameInfo::GetGameSizeInBytes() {
 	switch (fileType) {
 	case FILETYPE_PSP_PBP_DIRECTORY:
+	case FILETYPE_PSP_SAVEDATA_DIRECTORY:
+	{
+		std::vector<FileInfo> fileInfo;
+		getFilesInDir(filePath_.c_str(), &fileInfo);
+		int64_t sizeSum = 0;
+		// Note: getFileInDir does not fill in fileSize properly.
+		for (size_t i = 0; i < fileInfo.size(); i++) {
+			FileInfo finfo;
+			getFileInfo(fileInfo[i].fullName.c_str(), &finfo);
+			if (!finfo.isDirectory)
+				sizeSum += finfo.size;
+		}
 		// TODO: Need to recurse here.
-		return 0;
+		return sizeSum;
+	}
 	default:
 		return GetFileLoader()->FileSize();
 	}
 }
 
+// Not too meaningful if the object itself is a savedata directory...
 std::vector<std::string> GameInfo::GetSaveDataDirectories() {
 	std::string memc = GetSysDirectory(DIRECTORY_SAVEDATA);
 
@@ -117,6 +137,9 @@ std::vector<std::string> GameInfo::GetSaveDataDirectories() {
 }
 
 u64 GameInfo::GetSaveDataSizeInBytes() {
+	if (fileType == FILETYPE_PSP_SAVEDATA_DIRECTORY || fileType == FILETYPE_PPSSPP_SAVESTATE) {
+		return 0;
+	}
 	std::vector<std::string> saveDataDir = GetSaveDataDirectories();
 
 	u64 totalSize = 0;
@@ -132,7 +155,7 @@ u64 GameInfo::GetSaveDataSizeInBytes() {
 				filesSizeInDir += finfo.size;
 		}
 		if (filesSizeInDir < 0xA00000) {
-			//Generally the savedata size in a dir shouldn't be more than 10MB.
+			// HACK: Generally the savedata size in a dir shouldn't be more than 10MB.
 			totalSize += filesSizeInDir;
 		}
 		filesSizeInDir = 0;
@@ -141,6 +164,9 @@ u64 GameInfo::GetSaveDataSizeInBytes() {
 }
 
 u64 GameInfo::GetInstallDataSizeInBytes() {
+	if (fileType == FILETYPE_PSP_SAVEDATA_DIRECTORY || fileType == FILETYPE_PPSSPP_SAVESTATE) {
+		return 0;
+	}
 	std::vector<std::string> saveDataDir = GetSaveDataDirectories();
 
 	u64 totalSize = 0;
@@ -156,7 +182,7 @@ u64 GameInfo::GetInstallDataSizeInBytes() {
 				filesSizeInDir += finfo.size;
 		}
 		if (filesSizeInDir >= 0xA00000) { 
-			// Generally the savedata size in a dir shouldn't be more than 10MB.
+			// HACK: Generally the savedata size in a dir shouldn't be more than 10MB.
 			// This is probably GameInstall data.
 			totalSize += filesSizeInDir;
 		}
@@ -371,6 +397,33 @@ class GameInfoWorkItem : public PrioritizedWorkQueueItem {
 			}
 			break;
 
+		case FILETYPE_PSP_SAVEDATA_DIRECTORY:
+		{
+			SequentialHandleAllocator handles;
+			VirtualDiscFileSystem umd(&handles, gamePath_.c_str());
+
+			// Alright, let's fetch the PARAM.SFO.
+			std::string paramSFOcontents;
+			if (ReadFileToString(&umd, "/PARAM.SFO", &paramSFOcontents, 0)) {
+				lock_guard lock(info_->lock);
+				info_->paramSFO.ReadSFO((const u8 *)paramSFOcontents.data(), paramSFOcontents.size());
+				info_->ParseParamSFO();
+			}
+
+			ReadFileToString(&umd, "/ICON0.PNG", &info_->iconTextureData, &info_->lock);
+			info_->iconDataLoaded = true;
+			if (info_->wantFlags & GAMEINFO_WANTBG) {
+				ReadFileToString(&umd, "/PIC1.PNG", &info_->pic1TextureData, &info_->lock);
+				info_->pic1DataLoaded = true;
+			}
+			break;
+		}
+
+		case FILETYPE_PPSSPP_SAVESTATE:
+		{
+			break;
+		}
+
 		case FILETYPE_PSP_DISC_DIRECTORY:
 			{
 				info_->fileType = FILETYPE_PSP_ISO;
@@ -399,6 +452,7 @@ class GameInfoWorkItem : public PrioritizedWorkQueueItem {
 				}
 				break;
 			}
+
 		case FILETYPE_PSP_ISO:
 		case FILETYPE_PSP_ISO_NP:
 			{
@@ -517,7 +571,6 @@ class GameInfoWorkItem : public PrioritizedWorkQueueItem {
 };
 
 
-
 GameInfoCache::~GameInfoCache() {
 	Clear();
 }
@@ -531,18 +584,6 @@ void GameInfoCache::Shutdown() {
 	StopProcessingWorkQueue(gameInfoWQ_);
 }
 
-void GameInfoCache::Save() {
-	// TODO
-}
-
-void GameInfoCache::Load() {
-	// TODO
-}
-
-void GameInfoCache::Decimate() {
-	// TODO
-}
-
 void GameInfoCache::Clear() {
 	if (gameInfoWQ_)
 		gameInfoWQ_->Flush();

UI/GameInfoCache.h

@@ -33,6 +33,8 @@ class Thin3DTexture;
 // does on the PSP, namely checking for and deleting savedata, and similar things.
 // Only cares about games that are installed on the current device.
 
+// A GameInfo object can also represent a piece of savedata.
+
 // Guessed from GameID, not necessarily accurate
 enum GameRegion {
 	GAMEREGION_JAPAN,
@@ -94,12 +96,12 @@ class GameInfo {
 public:
 	GameInfo()
 		: disc_total(0), disc_number(0), region(-1), fileType(FILETYPE_UNKNOWN), paramSFOLoaded(false),
-		  iconTexture(NULL), pic0Texture(NULL), pic1Texture(NULL), wantFlags(0),
-		  timeIconWasLoaded(0.0), timePic0WasLoaded(0.0), timePic1WasLoaded(0.0),
+		  iconTexture(nullptr), pic0Texture(nullptr), pic1Texture(nullptr), wantFlags(0),
+		  lastAccessedTime(0.0), timeIconWasLoaded(0.0), timePic0WasLoaded(0.0), timePic1WasLoaded(0.0),
 		  gameSize(0), saveDataSize(0), installDataSize(0), fileLoader(nullptr) {}
 	~GameInfo();
 
-	bool DeleteGame();  // Better be sure what you're doing when calling this.
+	bool Delete();  // Better be sure what you're doing when calling this.
 	bool DeleteAllSaveData();
 	bool LoadFromPath(const std::string &gamePath);
 	FileLoader *GetFileLoader();
@@ -180,13 +182,8 @@ class GameInfoCache {
 	// redrawing the UI often. Only set flags to GAMEINFO_WANTBG or WANTSND if you really want them 
 	// because they're big. bgTextures and sound may be discarded over time as well.
 	GameInfo *GetInfo(Thin3DContext *thin3d, const std::string &gamePath, int wantFlags);
-	void Decimate();  // Deletes old info.
 	void FlushBGs();  // Gets rid of all BG textures. Also gets rid of bg sounds.
 
-	// TODO - save cache between sessions
-	void Save();
-	void Load();
-
 	PrioritizedWorkQueue *WorkQueue() { return gameInfoWQ_; }
 
 private:

UI/GameScreen.cpp

@@ -265,7 +265,7 @@ UI::EventReturn GameScreen::OnDeleteGame(UI::EventParams &e) {
 void GameScreen::CallbackDeleteGame(bool yes) {
 	GameInfo *info = g_gameInfoCache.GetInfo(NULL, gamePath_, 0);
 	if (yes) {
-		info->DeleteGame();
+		info->Delete();
 		g_gameInfoCache.Clear();
 		screenManager()->switchScreen(new MainScreen());
 	}