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

[Reimar Döffinger]

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.