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

[Istvan Sebok]

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?

Istvan

Samples:
https://www.dropbox.com/sh/306cn1sz3uwtkae/FbMe-YxweZ