Skip to content
Permalink
Browse files

Core: Generate screenshot name with timestamps instead of only increa…

…sing numbers
  • Loading branch information...
CookiePLMonster committed Aug 26, 2019
1 parent 71ff97c commit ff8f978eafb3a45959280a7a0c61bf11766dd040
Showing with 14 additions and 8 deletions.
  1. +14 −8 Source/Core/Core/Core.cpp
@@ -13,6 +13,7 @@
#include <variant>

#include <fmt/format.h>
#include <fmt/time.h>

#ifdef _WIN32
#include <windows.h>
@@ -677,15 +678,20 @@ static std::string GenerateScreenshotFolderPath()

static std::string GenerateScreenshotName()
{
std::string path = GenerateScreenshotFolderPath();

// append gameId, path only contains the folder here.
path += SConfig::GetInstance().GetGameID();
const std::string path_prefix =
GenerateScreenshotFolderPath() + SConfig::GetInstance().GetGameID();

const std::time_t cur_time = std::time(nullptr);
const std::string base_name =
fmt::format("{}_{:%Y-%m-%d_%H-%M-%S}", path_prefix, *std::localtime(&cur_time));

std::string name;
for (int i = 1; File::Exists(name = fmt::format("{}-{}.png", path, i)); ++i)
// First try a filename without any suffixes, if already exists then append increasing numbers
std::string name = fmt::format("{}.png", base_name);
if (File::Exists(name))
{
// TODO?
for (u32 i = 1; File::Exists(name = fmt::format("{}_{}.png", base_name, i)); ++i)
;
}

return name;
@@ -709,8 +715,8 @@ void SaveScreenShot(std::string_view name, bool wait_for_completion)

SetState(State::Paused);

const std::string path = fmt::format("{}{}.png", GenerateScreenshotFolderPath(), name);
g_renderer->SaveScreenshot(path, wait_for_completion);
g_renderer->SaveScreenshot(fmt::format("{}{}.png", GenerateScreenshotFolderPath(), name),
wait_for_completion);

if (!bPaused)
SetState(State::Running);

0 comments on commit ff8f978

Please sign in to comment.
You can’t perform that action at this time.