Re: [PATCH v3 0/11] MR557: mciqtz32: Correct video window behavior by creating default window.

[Akihiro Sagawa (＠sgwaki)]

On Tue Aug  2 05:14:10 2022 +0000, **** wrote:
> Zebediah Figura (she/her) replied on the mailing list:
> ```
> 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.
>
> wine-gitlab mailing list -- wine-gitlab(a)winehq.org
> To unsubscribe send an email to wine-gitlab-leave(a)winehq.org
> ```
> Got it. Good tip.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/557#note_5584