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

Ramiro Polla ramiro.polla
Sat May 30 20:24:02 CEST 2009


On Sat, May 30, 2009 at 1:28 PM, Michael Niedermayer <michaelni at gmx.at> 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

It would be nice to have this documented in doc/APIChanges.

Ramiro Polla



More information about the ffmpeg-devel mailing list