[PATCH:xf86-video-mga] Fix pointer cast warning.

Tormod Volden lists.tormod at gmail.com
Wed Feb 25 02:31:50 PST 2015


On Tue, Feb 24, 2015 at 7:36 AM, Mark Kettenis <mark.kettenis at xs4all.nl> wrote:
>>  cast to pointer from integer of different size
>>  [-Werror=int-to-pointer-cast]
>>
>>  Signed-off-by: Thomas Klausner <wiz at NetBSD.org>
>>  ---
>>   src/mga_exa.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> This code makes no sense to me.  exaGetPixmapFirstPixel() returns a pixel
> value.  Interpreting that as a pointer to the first pixel must be wrong.

Are you sure about this? What would be the utility of a function
returning the pixel /value/ of the first pixel in a pixmap?

>
> Note that the actual usage of mgaDownloadFromScreen() is #if 0'ed out.
> Perhaps this code should just be removed from the driver?  Or perhaps the
> code was #if 0'ed out because the code didn't work?

About the EXA code working or not, see also
https://bugs.freedesktop.org/show_bug.cgi?id=53423

Regards,
Tormod


More information about the xorg-devel mailing list