[FFmpeg-cvslog] r17404 - trunk/libavcodec/mpegvideo_xvmc.c

[Ivan Kalvachev]

On 2/18/09, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Wed, Feb 18, 2009 at 12:51:28AM +0100, diego wrote:
>> Author: diego
>> Date: Wed Feb 18 00:51:28 2009
>> New Revision: 17404
>>
>> Log:
>> Replace sizeof(short) by the more robust sizeof(*s->pblocks[i]) in memset
>> and
>> memcpy calls.  s->pblocks is the element actually being handled.
>
> ok +2 for you
>
> diego 19
> ivan 20.5

That's actually wrong. XvMC defines block data as short
and this is how it is in the xvmc structure.
pblocks points into the data array so sizeof(short) is the correct way to use

Now, making xvmc work when DCTELEM is not short, is interesting task.
But for now there is no reason for me to implement it.