[FFmpeg-devel] [PATCH][VAAPI][3/6] Add MPEG-2 bitstream decoding (take 6)
Mon Mar 23 11:23:38 CET 2009
On Sun, 22 Mar 2009, Gwenole Beauchesne wrote:
> Le 22 mars 09 ? 04:05, Michael Niedermayer a ?crit :
>>> +/** Is the current field the first field for field picture? */
>>> +static inline int mpeg2_get_is_first_field(MpegEncContext *s)
>>> + return s->first_field || s->picture_structure == PICT_FRAME;
>> picture_structure == PICT_FRAME are not field pics
> Actually, VA API first_field means !second_field and this is the same
> condition as mpeg_field_start()'s. Hmm, I will have to visually check for
> this one.
Yes, I confirm this, this needs to remain as is. i.e. first_field is
!second-field, which is
!(s->picture_structure != PICT_FRAME && !s->first_field), and thus
s->first_field || s->picture_structure == PICT_FRAME
New patch attached to fix other cosmetics too.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 8556 bytes
More information about the ffmpeg-devel