Re: [PATCH v2 08/11] mciqtz32: Correct initial video window dimensions.

[Zebediah Figura (she/her)]

On 7/31/22 06:53, Akihiro Sagawa wrote:
> @@ -183,9 +183,13 @@ static BOOL MCIQTZ_CreateWindow(WINE_MCIQTZ* wma, DWORD dwFlags, LPMCI_DGV_OPEN_
>       else if (FAILED(hr))
>           return FALSE;
>
> -    /* FIXME: Use video size here */
> -    width = 640;
> -    height = 480;
> +    /* Native always assumes an overlapped window
> +       when calculating default video window size. */
> +    SetRect(&rc, 0, 0, width, height);
> +    AdjustWindowRect(&rc, WS_OVERLAPPEDWINDOW, FALSE);
> +    min_width = GetSystemMetrics(SM_CXMIN);
> +    width = max(rc.right - rc.left, min_width);

CreateWindow() should automatically enforce the minimum width, so there 
should be no need to use SM_CXMIN here.