[Mesa-dev] [PATCH 3/3] xorg/xvmc: Add missing call to set_picture_parameters

Maarten Lankhorst m.b.lankhorst at gmail.com
Sun Aug 28 10:13:43 PDT 2011


Hey,

On 08/28/2011 05:04 PM, Christian König wrote:
> Hi Maarten,
>
> could you put this into SetDecoderStatus instead? This makes the picture
> structure also available in end_frame.
Erm it doesn't make sense there. It might make sense to move it to after
the begin_frame though, but with interlaced frames I don't see why you
would want to move it into SetDecoderStatus, since you could have interlaced
frames with different picture_structures.
> Also named initializers shouldn't be used according to the coding rules.
Ok. I'll fix that.

~Maarten


More information about the mesa-dev mailing list