[FFmpeg-devel] [PATCH] avcodec thread safety fix

Andreas Öman andreas
Sat May 30 20:08:45 CEST 2009


Michael Niedermayer wrote:
> On Sat, May 30, 2009 at 06:29:59PM +0200, Andreas ?man wrote:
>> Michael Niedermayer wrote:
>>> On Sat, May 30, 2009 at 02:03:01PM +0200, Andreas ?man wrote:
>>>> Michael Niedermayer wrote:
>>>>> indent ...
>>>> indeed ...
>>> [...]
>>>> @@ -657,6 +676,11 @@
>>>>      av_freep(&avctx->priv_data);
>>>>      avctx->codec = NULL;
>>>>      entangled_thread_counter--;
>>>> +
>>>> +    /* Release any user-supplied mutex. */
>>>> +    if (ff_lockmgr_cb) {
>>>> +      (*ff_lockmgr_cb)(&codec_mutex, AV_LOCK_RELEASE);
>>>> +    }
>>>>      return 0;
>>>>  }
>>> indent
>> Embarrassing.. Apart from this, is there anything else you want to have
>> fixed?
> 
> probably not, assuming noone else has any further comments

I'll wait for Art Clarke to see if he has anything to say then since he
seems to have some opinion in the subject.



More information about the ffmpeg-devel mailing list