[MPlayer-dev-eng] [PATCH] unified timing patch for H264

[Michael Niedermayer]

On Sat, Aug 21, 2010 at 08:24:35AM +0200, Reimar Döffinger wrote:
> On Fri, Aug 20, 2010 at 11:09:15PM +0200, Pásztor Szilárd wrote:
> > diff -NurpabB mplayer-export-2010-08-20/libavcodec/h264.c mplayer-export-2010-08-20-h264-timing/libavcodec/h264.c
> > --- mplayer-export-2010-08-20/libavcodec/h264.c	2010-07-05 16:36:03.000000000 +0200
> > +++ mplayer-export-2010-08-20-h264-timing/libavcodec/h264.c	2010-08-20 21:27:31.000000000 +0200
> > @@ -3021,7 +3021,7 @@ static int decode_frame(AVCodecContext *
> >  
> >          if (cur->field_poc[0]==INT_MAX || cur->field_poc[1]==INT_MAX) {
> >              /* Wait for second field. */
> > -            *data_size = 0;
> > +            *data_size = -1;
> 
> You can't use it like that, that's a size field and setting it like
> this would mean *data points to almost 4 GB of data - at least unless
> you change the API which requires a major verson bump, and that
> won't happen so soon.
> That's why I suggested changing the return value.

AVCodecContext.coded_frame.repeat_field
should tell you if the data you passed in was a field or a frame.
If it doesnt work, a fix is welcome

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

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20101006/bc4f7772/attachment-0001.pgp>