[FFmpeg-devel] [PATCH] CrystalHD: skip opaque_list_pop call when picture is a second field

Istvan Sebok sebok.istvan at gmail.com
Sun May 13 20:44:09 CEST 2012

2012/5/13 Philip Langdale <philipl at overt.org>

> On Sun, 13 May 2012 00:00:24 +0200
> Istvan Sebok <sebok.istvan at gmail.com> wrote:
> >
> This isn't right. In the case where an input field-pair
> gets broken up by the decoder, each output field will
> have the same opaque value associated with it. The first
> field will pop it off the list and the second will do
> nothing.
> In the case where there are two input fields, then each
> one will get a separate opaque value and, obviously, both
> should be popped to keep the list under control.
> OK, I understood this case. This patch isn't good.

> Did this change help you at all? You sent me another
> email saying the list was growing uncontrollably - which
> would happen with this change applied - as your samples
> include 1:2 and 2:2 field pairings.
> --phil

This problem exists. When I am playing DVB-T_1080i_CrystalHD_50M.ts file
the list is growing. For example between picture number 872 and 1146 the
list size is 68 and 334. Could you check it?



More information about the ffmpeg-devel mailing list