xf86-video-ati: [PATCH] Janitor: make distcheck, .gitignore.

Michel Dänzer michel at daenzer.net
Fri Feb 6 10:08:25 PST 2009


First of all, radeon driver development discussion takes place on the
xorg-driver-ati list. Moving there.

On Thu, 2009-02-05 at 21:23 -0200, Paulo César Pereira de Andrade wrote:
> I am starting to believe that, maybe -Wpointer-arith should
> not only be out of $CWARNFLAGS, but that $CWARNFLAGS should
> actually include -Wno-pointer-arith.
>   Almost every "casting" is a sign of something going wrong,
> as the compiler should always able to do proper type
> conversion automatically,

I made sure the driver builds for me with -Wall a while ago, and have
been building with -Wall -Werror since. I'm not sure I like the idea of
enabling even more warnings, especially as apparently none of them have
pointed out a real bug?

> and changing the void *'s in the ati driver to something like an
> "unsigned char *" just to correct that warning may not be the proper
> solution.

Are you referring to arithmetic on void* pointers? That's a gccism, so
if we have that we may want to fix it.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer



More information about the xorg mailing list