On 08/10/15 15:27, Derek Buitenhuis wrote:
> On 10/8/2015 9:08 AM, Rémi Denis-Courmont wrote:
>> I would assert and abort here. The most likely problems here are 
>> deadlocks (i.e. recursive initialization) and memory corruption (once 
>> ff_h264_vlc_init is garbage)...
> 
> I initially had this since Hendrik's patch could fail, but that is no
> longer the case.
> 
> I have no strong opinion on what to do in the error case - you're screwed
> regardless.
> 

Since, as you said, you cannot improve I'm keeping that part as-is and
I'd push the set if nobody has strong opinions (the no-thread wrapper
got the pointer dereferences fixed locally).

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to