Browse files

AVIDump: use dump_path rather than deprecated AVFormatContext::filename

AVFormatContext::filename was deprecated in lavf 58.7.100 in favor
of AVFormatContext::url. Instead of adding version-checking logic,
just use the passed-in dump path instead.
  • Loading branch information...
ligfx committed Jul 14, 2018
1 parent e45698a commit 857138a0ba051152529b8695fe775924c857f908
Showing with 5 additions and 5 deletions.
  1. +5 −5 Source/Core/VideoCommon/AVIDump.cpp
@@ -224,16 +224,16 @@ bool AVIDump::CreateVideoFile()
return false;
NOTICE_LOG(VIDEO, "Opening file %s for dumping", s_format_context->filename);
if (avio_open(&s_format_context->pb, s_format_context->filename, AVIO_FLAG_WRITE) < 0 ||
NOTICE_LOG(VIDEO, "Opening file %s for dumping", dump_path.c_str());
if (avio_open(&s_format_context->pb, dump_path.c_str(), AVIO_FLAG_WRITE) < 0 ||
avformat_write_header(s_format_context, nullptr))
ERROR_LOG(VIDEO, "Could not open %s", s_format_context->filename);
ERROR_LOG(VIDEO, "Could not open %s", dump_path.c_str());
return false;
OSD::AddMessage(StringFromFormat("Dumping Frames to \"%s\" (%dx%d)", s_format_context->filename,
s_width, s_height));
StringFromFormat("Dumping Frames to \"%s\" (%dx%d)", dump_path.c_str(), s_width, s_height));
return true;

0 comments on commit 857138a

Please sign in to comment.