xf86-video-ati: Branch 'master'

[Michel Dänzer]

On Wed, 2008-02-20 at 11:45 +0200, Daniel Stone wrote:
> On Wed, Feb 20, 2008 at 10:28:32AM +0100, Michel Dänzer wrote:
> > I'd like to once again encourage everybody to build with -Wall
> > -Werror ... with recent versions of gcc, you may also need
> > -Wno-error=address to avoid erroring out on a warning from the
> > REGION_INIT() macro that I haven't figured out a reasonable way to
> > avoid.
> 
> If it warns on pScreen being unused because of this:
> ScreenPtr pScreen = something->pScreen;
> REGION_INIT(pScreen, ...);
> 
> then you can use this instead:
> ScreenPtr pScreen;
> pScreen = something->pScreen;
> REGION_INIT(pScreen, ...);

Thanks, but it's (sorry, should have pasted this before):

../../src/radeon_video.c: In function 'radeon_crtc_clip_video_helper':
../../src/radeon_video.c:216: warning: the address of 'crtc_box' will always evaluate as 'true'

due to the if (_rect) test in the macro.


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