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 -- [e-mail]
> To unsubscribe send an email to [e-mail]

Got it. Good tip.