[PATCH xserver] Make PixmapDirtyUpdateRec::src a DrawablePtr

2017-04-18 Thread Michel Dänzer
From: Michel Dänzer This allows making the master screen's pixmap_dirty_list entries explicitly reflect that we're now tracking the root window instead of the screen pixmap, in order to allow Present page flipping on master outputs while there are active slave outputs.

[PATCH xf86-video-amdgpu] Adapt to PixmapDirtyUpdateRec::src being a DrawablePtr

2017-04-18 Thread Michel Dänzer
From: Michel Dänzer Signed-off-by: Michel Dänzer --- Chris / Ilia / Ben, this should be manageable for the intel/nouveau drivers, right? src/amdgpu_drv.h | 26 ++ src/amdgpu_kms.c | 18 +-

[PATCH xserver 3/4] hw/xwin: Fix -Wmaybe-uninitialized warnings in engine CloseScreen wrappers

2017-04-18 Thread Jon Turney
../hw/xwin/winshadgdi.c: In function ‘winCloseScreenShadowGDI’: ../hw/xwin/winshadgdi.c:632:12: warning: ‘fReturn’ may be used uninitialized in this function [-Wmaybe-uninitialized] ../hw/xwin/winshadgdi.c:579:10: note: ‘fReturn’ was declared here ../hw/xwin/winshadddnl.c: In function

[PATCH xserver 4/4] hw/xwin: Fix -Wunused-function warning for MessageName() in non-debug builds

2017-04-18 Thread Jon Turney
../hw/xwin/winmultiwindowwm.c:188:1: warning: ‘MessageName’ defined but not used [-Wunused-function] Signed-off-by: Jon Turney --- hw/xwin/winmultiwindowwm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/xwin/winmultiwindowwm.c

[PATCH xserver 1/4] Don't declare variables which will be unused if XSERVER_LIBPICACCESS isn't defined

2017-04-18 Thread Jon Turney
../hw/xfree86/common/xf86Config.c: In function ‘xf86HandleConfigFile’: ../hw/xfree86/common/xf86Config.c:2278:10: warning: unused variable ‘singlecard’ [-Wunused-variable] ../hw/xfree86/common/xf86Config.c:2277:17: warning: unused variable ‘scanptr’ [-Wunused-variable] Signed-off-by: Jon Turney

[PATCH xserver 0/4] Warning fixes

2017-04-18 Thread Jon Turney
Experimenting with building with meson, it likes to use a different set of warning flags by default, which make the following a bit more obvious. Jon Turney (4): Don't declare variables which will be unused if XSERVER_LIBPICACCESS isn't defined hw/xwin: Fix -Wmaybe-uninitialized warning

[PATCH xserver 2/4] hw/xwin: Fix -Wmaybe-uninitialized warning in winWindowProc

2017-04-18 Thread Jon Turney
This is possibly an actual bug in failing to check we successfuly retrieved the monitor size before using it to set the X screen size. ../hw/xwin/winwndproc.c: In function ‘winWindowProc’: ../hw/xwin/winwndproc.c:283:55: warning: ‘dwHeight’ may be used uninitialized in this function

Re: [PATCH xserver 2/4] hw/xwin: Fix -Wmaybe-uninitialized warning in winWindowProc

2017-04-18 Thread Adam Jackson
On Tue, 2017-04-18 at 11:53 +0100, Jon Turney wrote: > This is possibly an actual bug in failing to check we successfuly retrieved > the monitor size before using it to set the X screen size. It is indeed. > @@ -261,6 +261,9 @@ winWindowProc(HWND hwnd, UINT message, WPARAM wParam, > LPARAM