[FFmpeg-devel] [RFC] support for Cam_PAL_D1_inter_Field_25fps_2.5MBS.mp4

[Michael Niedermayer]

On Wed, Mar 26, 2008 at 09:57:35AM +0100, Reimar D?ffinger wrote:
> Hello,
> On Wed, Mar 26, 2008 at 03:46:59AM +0100, Michael Niedermayer wrote:
> > On Fri, Mar 21, 2008 at 01:50:47PM +0100, Reimar D?ffinger wrote:
> > > @@ -7600,6 +7614,10 @@
> > >               * NAL unit stuff to context 0 and restart. Note that
> > >               * rbsp_buffer is not transfered, but since we no longer
> > >               * run in parallel mode this should not be an issue. */
> > > +            if(context_count) {
> > > +                execute_decode_slices(h, context_count);
> > > +                context_count = 0;
> > > +            }
> > 
> > What values do context_count and h->max_contexts have here? and why
> > does the 
> > if(context_count == h->max_contexts) {
> > above not trigger?
> > 
> > max_contexts should be 1 and context_count should be 1 as well i think
> 
> In this case yes, but my idea was that the code should also work and
> decode in parallel if each field is coded into multiple slices.
> Since I have no actual file to test that, I could have done nonsense I
> admit...

I assume that all the other changes are actually needed or are there further
that are just there because they could do some good in some hyphothetical case?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080401/56a30bc1/attachment.pgp>