From e491f8cd485f5602c056b8b6c5e0c8c395e4a633 Mon Sep 17 00:00:00 2001 From: ShadowNinja Date: Sun, 1 Jun 2014 13:11:37 -0400 Subject: Only try to load from possible player files --- src/player.cpp | 61 ++++++++++++++++++++-------------------------------------- 1 file changed, 21 insertions(+), 40 deletions(-) (limited to 'src/player.cpp') diff --git a/src/player.cpp b/src/player.cpp index 78ba17e89..a52385f20 100644 --- a/src/player.cpp +++ b/src/player.cpp @@ -284,11 +284,10 @@ void Player::clearHud() } -void RemotePlayer::save(const std::string &savedir) +void RemotePlayer::save(std::string savedir) { - bool newplayer = true; - - /* We have to iterate through all files in the players directory + /* + * We have to open all possible player files in the players directory * and check their player names because some file systems are not * case-sensitive and player names are case-sensitive. */ @@ -296,23 +295,25 @@ void RemotePlayer::save(const std::string &savedir) // A player to deserialize files into to check their names RemotePlayer testplayer(m_gamedef); - std::vector player_files = fs::GetDirListing(savedir); - for(u32 i = 0; i < player_files.size(); i++) { - if (player_files[i].dir || player_files[i].name[0] == '.') { - continue; + savedir += DIR_DELIM; + std::string path = savedir + m_name; + for (u32 i = 0; i < 1000; i++) { + if (!fs::PathExists(path)) { + // Open file and serialize + std::ostringstream ss(std::ios_base::binary); + serialize(ss); + if (!fs::safeWriteToFile(path, ss.str())) { + infostream << "Failed to write " << path << std::endl; + } + return; } - - // Full path to this file - std::string path = savedir + "/" + player_files[i].name; - // Open file and deserialize std::ifstream is(path.c_str(), std::ios_base::binary); if (!is.good()) { - infostream << "Failed to read " << path << std::endl; - continue; + infostream << "Failed to open " << path << std::endl; + return; } - testplayer.deSerialize(is, player_files[i].name); - + testplayer.deSerialize(is, path); if (strcmp(testplayer.getName(), m_name) == 0) { // Open file and serialize std::ostringstream ss(std::ios_base::binary); @@ -320,33 +321,13 @@ void RemotePlayer::save(const std::string &savedir) if (!fs::safeWriteToFile(path, ss.str())) { infostream << "Failed to write " << path << std::endl; } - newplayer = false; - break; - } - } - - if (newplayer) { - bool found = false; - std::string path = savedir + "/" + m_name; - for (u32 i = 0; i < 1000; i++) { - if (!fs::PathExists(path)) { - found = true; - break; - } - path = savedir + "/" + m_name + itos(i); - } - if (!found) { - infostream << "Didn't find free file for player " << m_name << std::endl; return; } - - // Open file and serialize - std::ostringstream ss(std::ios_base::binary); - serialize(ss); - if (!fs::safeWriteToFile(path, ss.str())) { - infostream << "Failed to write " << path << std::endl; - } + path = savedir + m_name + itos(i); } + + infostream << "Didn't find free file for player " << m_name << std::endl; + return; } /* -- cgit v1.2.3