On Mon, Apr 15, 2013 at 12:42 PM, Luca Barbato <[email protected]> wrote:
> On 04/11/2013 04:56 PM, Yusuke Nakamura wrote: > > 2013/4/9 Luca Barbato <[email protected]> > > > >> On 09/04/13 10:16, Yusuke Nakamura wrote: > >>> Attached patches support setting field order information for H.264, > VC-1 > >>> and MPEG-2 video stream by parsing. > >> > >> They look ok, the h264 one might be reformatted a little since chained > >> ternaries aren't that nice to follow. > >> > >> lu > >> _______________________________________________ > >> libav-devel mailing list > >> [email protected] > >> https://lists.libav.org/mailman/listinfo/libav-devel > >> > > > > > > I replaced chained ternaries with if-else. > > > > I like the idea, who is against it? > > lu > Unless I'm missing something and if my understanding of the spec is correct, the SEI message used here to update field_order is originally intended only as a hint for the display. The "correct" way to understand if frame has been coded interlaced or progressive is by looking at frame_mbs_only_flag and field_pic_flag. Then again somewhere in the spec it should be written that the SEI must be initialized to the same value of frame_mbs_only_flag but I wouldn't rely on an optional SEI message to get the field order information. Best, Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
