[FFmpeg-devel] [PATCH] H.264 fix interlaced flag: bring back pic_struct

[Haruhiko Yamagata]

On Tue, Mar 03, 2009, Michael Niedermayer wrote:
> ct_type should always be used if available, otherwise
> FIELD_OR_MBAFF_PICTURE should be.
> that at least is how i understand the spec, if this fails for some
> case iam interrested in the file

Excuse me for late response, I found a file which fails.
http://x264.nl/h.264.samples/premiere-paff.ts

Please note that ct_type is optional in the picture timing SEI.

>            if(get_bits(&s->gb, 1)){                  /* clock_timestamp_flag */
>                unsigned int full_timestamp_flag;
>                h->sei_ct_type |= 1<<get_bits(&s->gb, 2);

That file does not have ct_type at all, but has correct pic_struct.
pic_struct is basically correct, only a few streams are badly encoded.
The attached patch fixes this issue and still honor ct_type if available.

Best regards,
Haruhiko Yamagata
-------------- next part --------------
A non-text attachment was scrubbed...
Name: check_presence_of_cy_type.patch
Type: application/octet-stream
Size: 2235 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090528/16dd66d5/attachment.obj>