Fix saving screenshots of dummy video

This commit is contained in:
Thomas Goyne 2013-10-16 17:29:03 -07:00
parent e57035c832
commit 7bd1da7d45

View file

@ -461,14 +461,16 @@ struct video_frame_prev_large : public validator_video_loaded {
}; };
static void save_snapshot(agi::Context *c, bool raw) { static void save_snapshot(agi::Context *c, bool raw) {
static const agi::OptionValue* ssPath = OPT_GET("Path/Screenshot"); auto option = OPT_GET("Path/Screenshot")->GetString();
std::string option = ssPath->GetString();
agi::fs::path basepath; agi::fs::path basepath;
auto videoname = c->videoController->GetVideoName();
bool is_dummy = boost::starts_with(videoname.string(), "?dummy");
// Is it a path specifier and not an actual fixed path? // Is it a path specifier and not an actual fixed path?
if (option[0] == '?') { if (option[0] == '?') {
// If dummy video is loaded, we can't save to the video location // If dummy video is loaded, we can't save to the video location
if (boost::starts_with(option, "?video") && boost::starts_with(c->videoController->GetVideoName().string(), "?dummy")) { if (boost::starts_with(option, "?video") && is_dummy) {
// So try the script location instead // So try the script location instead
option = "?script"; option = "?script";
} }
@ -483,7 +485,8 @@ static void save_snapshot(agi::Context *c, bool raw) {
// Actual fixed (possibly relative) path, decode it // Actual fixed (possibly relative) path, decode it
else else
basepath = config::path->MakeAbsolute(option, "?user/"); basepath = config::path->MakeAbsolute(option, "?user/");
basepath /= c->videoController->GetVideoName().stem();
basepath /= is_dummy ? "dummy" : videoname.stem();
// Get full path // Get full path
int session_shot_count = 1; int session_shot_count = 1;