[FFmpeg-devel] [PATCH] Fix roq encoding

[Vitor Sessak]

Michael Niedermayer wrote:
> On Sat, Jul 19, 2008 at 04:25:17AM +0200, Vitor Sessak wrote:
>> Michael Niedermayer wrote:
>>> On Sat, Jul 19, 2008 at 12:23:28AM +0200, Vitor Sessak wrote:
>>>> Hi,
>>>>
>>>> The following patch fix the issue 548. Not that I fully understand why
>>>> before it was ok not to set avctx->coded_frame and now it crashes...
>>>>
>>>> -Vitor
>>>>
>>>> PS: nice to know there is someone who used this encoder...
>>>> Index: libavcodec/roqvideoenc.c
>>>> ===================================================================
>>>> --- libavcodec/roqvideoenc.c	(revision 14271)
>>>> +++ libavcodec/roqvideoenc.c	(working copy)
>>>> @@ -1037,6 +1037,8 @@
>>>>      /* Encode the actual frame */
>>>>      roq_encode_video(enc);
>>>>  +    avctx->coded_frame = enc->last_frame;
>>> I think it should be set to current_frame before the pointers
>>> are swapped, that would be cleaner
>> I agree, like the attached patch?
> 
> yes, patch ok

Applied.

-Vitor