On Mon, Sep 14, 2009 at 10:20:57PM +0200, Carlos R. Mafra wrote:
> and I was wondering why it is necessary for 64-bits machines.
This patch sucks.

> But why do we want to change 'data' and 'pixel' for 64-bits machines?
We don't.  'data' contains width, height and pixels.  width and
height defined in RImage as 'int'.  Then 'data' is passed to
XChangeProperty() as list of 32-bit quantities which is not true
on 64-bit.

>      data[o++] = image->width;
Better make sure that image->width and image->height fits to
CARD32 here and on next line.

-- 
Regards,
Sir Raorn.

Attachment: signature.asc
Description: Digital signature

Reply via email to