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

[Michael Niedermayer]

On Tue, Apr 01, 2008 at 10:15:00AM +0200, Reimar D?ffinger wrote:
> On Tue, Apr 01, 2008 at 04:02:40AM +0200, Michael Niedermayer wrote:
> > Anyway, i think the following might be a fix, not to h264 being a mess but
> > the actual issue here (Note i do not claim to fully
> > understand the problem so what i say could be stupid)
> > 
> > decode_slice_header() should check for the case of being feeded a second
> > field without having the first done, it should return a special error code
> > for that case.
> > 
> > the code currently around the call to execute_ref_pic_marking() should
> > be moved somewhere else, like im its own function. And this should be
> > called from decode_nal_units() "when a field is done".
> 
> I will try to find time for it if nobody else does and supporting it in
> the decoder instead of an AVParser is the preferred way.

I think we should support it in the parser but that doesnt mean the decoder
should be unflexible.
For the case of .mp4 a parser should be able to split the NALs quickly but
for annexB it would be faster to handle it in the decoder.

[...]
-- 
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: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080401/39812c70/attachment.pgp>