Page 1 of 1

Assert in FFmpegLibs::InitLibs

Posted: Tue Nov 11, 2014 6:11 pm
by Edgar
This is been around for a while, is it just me or do other folks see it when running under the debugger (I am not testing the wxWidgets 3 configuration). In srcFFmpeg.cpp at or near line number 849, in the function:
bool FFmpegLibs::InitLibs(wxString libpath_format, bool WXUNUSED(showerr))

Code: Select all

      wxLogMessage(wxT("Actual avutil path %s"),
                 FileNames::PathFromAddr(avutil->GetSymbol(wxT("avutil_version"))).c_str());
causes an assertion failure:
assert.png
assert.png (59.76 KiB) Viewed 1396 times
Since this debugging code has languished for many months, does it make any sense to comment out these debugging messages:
code.png
code.png (43.49 KiB) Viewed 1396 times
to suppress the assert?

Re: Assert in FFmpegLibs::InitLibs

Posted: Wed Nov 12, 2014 1:46 pm
by Gale Andrews
Ed, are you saying this assert occurs even if FFmpeg is correctly installed (F,C,U numbers are visible in Libraries Preferences)?

Gale

Re: Assert in FFmpegLibs::InitLibs

Posted: Wed Nov 12, 2014 3:35 pm
by Edgar
Gale Andrews wrote:Ed, are you saying this assert occurs even if FFmpeg is correctly installed (F,C,U numbers are visible in Libraries Preferences)?
No, it only happens if you're trying to use Audacity without/before installing (the optional) FFmpeg library. Given that "without" & "before" are certain to happen (especially for newbies), this code silently blows up in the Release configuration and that this is unnecessary (and probably superfluous now) debugging code - I contend that it should be removed.