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

[Michael Niedermayer]

On Fri, Aug 20, 2010 at 08:17:29AM +0200, Reimar Döffinger wrote:
> On Thu, Aug 19, 2010 at 10:47:58PM -0700, Alexander Strange wrote:
> > > Yes it is clearly a bug, as vital information gets lost. However, in
> > > libavcodec/h264.c, if we return -1 instead of 0 (at "Wait for second
> > > field"),
> > > which may not be a pretty way of signalling this special case, it works.
> > 
> > I really don't want to see an error code returned for something not an error.
> 
> But it is an error, FFmpeg requires the data for a full frame for all decoders.
> As such, if there is only a single field in a packet, the decoder should return
> an error (preferably a special value meaning "well, I can handle this, but this
> is insufficient data").

This is incorrect, the h264 decoder as well as others can work with a slice at
a time if you set the correct flags.

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

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/77c6ba52/attachment.pgp>