xf86-video-ati: [PATCH] Janitor: make distcheck, .gitignore.
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
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