This change sounds wrong to me too.

On 5 June 2016 at 08:26, Thomas Faber <thomas.faber@reactos.org> wrote:
On 2016-06-05 01:29, hbelusca@svn.reactos.org wrote:
> -    rcDesk.left = GetSystemMetrics(SM_XVIRTUALSCREEN);
> -    rcDesk.top = GetSystemMetrics(SM_YVIRTUALSCREEN);
> -    rcDesk.right = rcDesk.left + GetSystemMetrics(SM_CXVIRTUALSCREEN);
> -    rcDesk.bottom = rcDesk.top + GetSystemMetrics(SM_CYVIRTUALSCREEN);
> +    rcDesk.left   = GetSystemMetrics(SM_XVIRTUALSCREEN);
> +    rcDesk.top    = GetSystemMetrics(SM_YVIRTUALSCREEN);
> +    rcDesk.right  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
> +    rcDesk.bottom = GetSystemMetrics(SM_CYVIRTUALSCREEN);

Are you sure about this? "CX" normally means width, so right = left + cx
seems more logical. And SM_XVIRUTALSCREEN is probably 0 most of the
time so it would be hard to notice.

_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev