[FFmpeg-devel] [PATCH] Fix return value for incomplete H264 frame packets

Pásztor Szilárd don
Mon Aug 23 15:54:54 CEST 2010


Carl Eugen Hoyos:
> I believe I already explained (on mplayer-dev-eng) that this patch makes
> ffmpeg print one warning message for every second PAFF field, so I don't
> think it is acceptable as-is.

Sorry, I didn't read your next mail before sending mine.
On mplayer-dev-eng we kind of agreed that lavc should sign somehow that it
received an incomplete frame (first field only, that is).
Still something not right here :(

Laurent Aimar:
>  Unless I misunderstood the current code, avcodec will consume the first
> field, properly decode it and return no pictures and no errors. If so, it
> seems the right behavior to me.

You understood it correctly. The question is if it should be a task of the
decoder to call for more data, or recognizing an incomplete frame in a packet
should be done in the application feeding data to the decoder. VLC and ffplay
somehow do the latter. I'll look into that then.

s.

                    ----------------------------------------
                    |  Empty heads have better acoustics.  |
                    ----------------------------------------



More information about the ffmpeg-devel mailing list