[FFmpeg-devel] [PATCH][VAAPI][3/6] Add MPEG-2 bitstream decoding (take 6)

[Gwenole Beauchesne]

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...
Name: ffmpeg.hwaccel.vaapi.mpeg2.6.patch
Type: text/x-diff
Size: 8556 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090323/fa911d0c/attachment.patch>