[FFmpeg-devel] [PATCH] H.264 fix interlaced flag

[Michael Niedermayer]

On Wed, Feb 18, 2009 at 10:22:34PM +0100, Ivan Schreter wrote:
> Hi Michael,
> 
> Ivan Schreter wrote:
> > Michael Niedermayer wrote:
> >   
> >> On Sun, Feb 15, 2009 at 08:23:31PM +0100, Ivan Schreter wrote:
> >>   
> >>     
> >>> In non-interlaced videos coded with picture struct top/bottom or 
> >>> bottom/top, the picture is marked as interlaced. Attached patch corrects 
> >>> it.
> >>> [...]
> >>>       
> >> please upload a sample that your patch fixes
> >>   
> >>     
> > Uploaded to dir h264_progressive_deemed_interlaced.
> >
> > However, I cannot upload description, it says "553 Could not create 
> > file". Retrying in new FTP session won't let me even cd to the directory 
> > (550 Failed to change directory). So it's without extra text file :-(
> >
> >   
> Did you have a chance to review it and eventually test it with submitted 
> file? I hope the file was uploaded correctly...

the patch is wrong, there seems no relation between the pic_struct and
the interlacing vs progressive.
Maybe the ct_type field could be used and the whole pic_struct->interlaced
code removed

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20090220/43e9b7d6/attachment.pgp>