Re: [PATCH 2/2] user32: scrollbars are vertical by default

[Nikolay Sivov]

On 04.03.2017 23:29, Kees Beets wrote:
> Fixes https://bugs.winehq.org/show_bug.cgi?id=38029
>
> Changed calculation of boolean vertical so that default is true.
>
> Tested on Ubuntu 16.04.
> Signed-off-by: [e-mail]
>
> dlls/user32/scroll.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/user32/scroll.c b/dlls/user32/scroll.c
> index 4b572e5..202fac8 100644
> --- a/dlls/user32/scroll.c
> +++ b/dlls/user32/scroll.c
> @@ -252,7 +252,7 @@ static BOOL SCROLL_GetScrollBarRect( HWND hwnd, INT nBar, RECT *lprect,
>    case SB_CTL:
>
> GetClientRect( hwnd, lprect );
> -
>    vertical = ((wndPtr->dwStyle & SBS_VERT) != 0);
> -
>    vertical = ((wndPtr->dwStyle & SBS_HORZ) == 0);
>
> break;
>
> default:

Hi,

this doesn't look right, SBS_HORZ is 0, which makes 'vertical' always 1.