x11 fullscreen mode seems to be broken in 12ec6c6

[arcadoss]

➜ FreeRDP git:(12ec6c6) git bisect bad 12ec6c6
12ec6c6 is the first bad commit
commit 12ec6c6
Author: Marc-André Moreau marcandre.moreau@gmail.com
Date: Sun Apr 28 18:48:27 2013 -0400

xfreerdp: add support for selecting individual monitors

:040000 040000 df5f378eb4ec98094759651efcc62cf774509980 925af87143a3abf917bc939785168c9e277b19ec M client
:040000 040000 9a0c655efc427387b2e46535b401c6044fc9ae61 b03323c7537347fb74cb1b6cd46f37b3ba359658 M include

[bailli]

It seems this bug depends on the X environment. I have encoutered some cases were fullscreen works and some were it doesn't.

A quick "hack" to get it working (but probably break individial monitors) is to change the following line 147 in xf_window.c (

FreeRDP/client/X11/xf_window.c

Line 147 in 12ec6c6

XMoveResizeWindow(xfi->display, window->handle, settings->DesktopPosX, settings->DesktopPosY, window->width, window->height);

)
settings->DesktopPosX and DesktopPosY seem to hold wrong values, changing these two parameters back to 0 (as before the mentioned commit) full screen works again (for me).

[bmiklautz]

I think this might also be related to #1695

[akallabeth]

Working on target release.