2013/4/16 Vittorio Giovara <vittorio.giov...@gmail.com>

> On 15/apr/2013, at 16:43, Yusuke Nakamura <muken.the.vfrman...@gmail.com>
> wrote:
>
> > As far as I understand, frame_mbs_only_flag == 1 guarantees that frames
> are
> > progressive, while frame_mbs_only_flag == 0 doesn't guarantee that a
> frame
> > is coded as progressive or interlaced.
> That's correct, that's why I mentioned field_pic_flag too. When such
> flag is set you should also check if tff or bff is set too.
>

Check and?
Do you want me to override field order by bottom_field_flag or output
something as log if mismatched?

> And, if CpbDpbDelaysPresentFlag is equal to 1 or pic_struct_present_flag
> is
> > equal to 1, Picture timing SEI is not optional and it shall be present in
> > every access unit.
> You are right, when pic_struct_present_flag is 1 it is mandatory to
> include the SEI message, however it is not mandatory to set
> pic_struct_present_flag at all.
> So it is possible to run into interlaced/pulled down streams without
> such SEI. In such cases only frame_mbs_only_flag and field_pic_flag
> will be able to correctly initialize avfieldorder (again, if my
> understanding of the spec is correct).
>

I can't grasp what you want to say.
I already initialized AVFieldOrder by frame_mbs_only_flag, field_pic_flag,
bottom_field_flag and POC when pic_struct_present_flag is 0 according to
the spec.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to