amejia1/xbmc — comment on commit 846f0b995d2dbe01856e378472be62479e461cb5
File:  line 0
By anssih:

I guess using a single critical section would cause a problem if FFmpeg has multiple threads trying to lock different mutexes. I guess that is somewhat unlikely, though, as it is currently only used in init/close paths and FFmpeg actually uses platform mutexes directly in all other places (I guess this lockmgr might be some legacy stuff).

In any case, I'd think av_lockmgr_register() call belongs to library load time, not in CDVDDemuxFFmpeg (see av_log_set_callback() in DllAvUtil.h).

Also, how did you determine av_find_stream_info() no longer requires locking?