On Sun, Oct 4, 2015 at 12:47 PM, Marton Balint <c...@passwd.hu> wrote: > > > On Sun, 4 Oct 2015, Ganesh Ajjanagadde wrote: > >> SDL_CreateMutex can fail: >> https://wiki.libsdl.org/SDL_CreateMutex. >> This patch makes creation more robust in one instance. >> >> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> >> --- >> ffplay.c | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/ffplay.c b/ffplay.c >> index c5a7895..84bd6ee 100644 >> --- a/ffplay.c >> +++ b/ffplay.c >> @@ -2817,6 +2817,12 @@ static int read_thread(void *arg) >> int scan_all_pmts_set = 0; >> int64_t pkt_ts; >> >> + if (!wait_mutex) { >> + av_log(is, AV_LOG_FATAL, "SDL_CreateMutex(): %s\n", >> SDL_GetError()); > > > VideoState has no AVClass, are you sure this works? All other instances use > NULL.
Entirely possible; I just (naively) assumed logging does some magic e.g void* pointers or something like that. You can use v1 then. > > Regards, > Marton > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
