[FFmpeg-devel] [PATCH] Fix encoding when delay > number of frames

Baptiste Coudurier baptiste.coudurier
Wed May 19 22:48:51 CEST 2010

On 03/29/2010 12:33 PM, Reimar D?ffinger wrote:
> On Mon, Mar 29, 2010 at 12:22:43PM -0700, Baptiste Coudurier wrote:
>> On 03/29/2010 11:22 AM, Reimar D?ffinger wrote:
>>> On Mon, Mar 29, 2010 at 11:06:26AM -0700, Baptiste Coudurier wrote:
>>>>> Also I think that our x264 encoder still does not use x264_encoder_delayed_frames,
>>>>> sure that isn't the real issue maybe?
>>>> That is the issue, but I'm not sure how it would be useful here.
>>>> Problem is:
>>>> All frames are given to the encoder in the main loop, no frames are
>>>> output in the main because delay is too big.
>>>> The flush loop outputs no frame because of the<= 0 return value check.
>>> The FFmpeg API is that way that an encoder should never return 0 if there
>>> are still delayed frames.
>>> So the libx264 module should loop itself until either it has some new data
>>> or there are no more delayed frames.
>> Right, patch attached.
> Looks right to me, but I am not the maintainer.

DS said it was fine so applied.

Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org

More information about the ffmpeg-devel mailing list