- wine-gitlab - winehq.org

Re: [PATCH v2 0/1] MR579: dwrite: Fix stretch value validation in CreateTextFormat(). - approved
by Alexandre Julliard (＠julliard) 03 Aug '22

03 Aug '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/579

1 0

Re: [PATCH v5 0/4] MR565: Implement NCryptEncrypt and BCryptEncrypt - approved
by Alexandre Julliard (＠julliard) 03 Aug '22

03 Aug '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/565

1 0

Re: [PATCH v4 0/2] MR511: webservices: Support error strings and faults in error. - approved
by Alexandre Julliard (＠julliard) 03 Aug '22

03 Aug '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/511

1 0

Re: [PATCH v2 0/19] MR578: d2d1: Add initial command list recording implementation. - approved
by Alexandre Julliard (＠julliard) 03 Aug '22

03 Aug '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/578

1 0

[PATCH 0/1] MR573: winebus.sys/sdl: Use the 8 absolute axes supported by DIJOYSTATE2.
by Arek Hiler (＠ivyl) 03 Aug '22

03 Aug '22

Most games use c_dfDIJoystick2 which can support up to 8 absolute axes, 6 directional ones and 2 sliders. With this change we now have 8 usable axes instead of six and we FIXME if device exposes more than that. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/573

3 3

[PATCH v9 0/10] MR391: scrrun: Implement MoveFolder().
by Robert Wilhelm (＠sloper42) 03 Aug '22

03 Aug '22

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52128 Signed-off-by: Robert Wilhelm <robert.wilhelm(a)gmx.net> -- v9: scrrun: Return path not found error if no folders were moved in MoveFolder(). scrrun: return path not found error if source ends with path separator in MoveFolder(). scrrun: Move directories only in MoveFolder(). scrrun: Support wildcards in MoveFolder(). scrrun: Move source dir into destination dir if destination ends with separator in … [View More]

3 11

[PATCH v3 0/4] MR393: wmvcore: Asynchronous reader state transitions.
by Rémi Bernon (＠rbernon) 03 Aug '22

03 Aug '22

Before https://gitlab.winehq.org/wine/wine/-/merge_requests/140, and to avoid a situation in the future where we might decide to implement this, and where the callback would be called asynchronously as they are supposed to, without the ASF reader filter supporting it. -- v3: winegstreamer: Make IWMReader state transitions asynchronous. winegstreamer: Leave the stream CS while waiting for a sample. winegstreamer: Move IWMReaderCallbackAdvanced *callback_advanced to a local … [View More]

3 5

Re: [PATCH v3 0/11] MR557: mciqtz32: Correct video window behavior by creating default window.
by Akihiro Sagawa (＠sgwaki) 03 Aug '22

03 Aug '22

On Tue Aug 2 05:14:16 2022 +0000, **** wrote: > Zebediah Figura (she/her) replied on the mailing list: > ``` > On 7/31/22 06:53, Akihiro Sagawa wrote: > > @@ -1573,6 +1575,37 @@ static void test_video_window(void) > > todo_wine_if(i != 3) > > ok(style == expected, "hwnd %p: got %#lx, expected %#lx\n", > video_window, style, expected); > > > > + /* get source video size */ > > + err = mciSendCommandW(wDeviceID, … [View More]MCI_WHERE, > MCI_DGV_WHERE_SOURCE, (DWORD_PTR)&parm); > > + ok(!err,"mciCommand where source returned %s\n", dbg_mcierr(err)); > > + src_rc = parm.where.rc; > This should just be equal to the whole video rect (0,0)-(64,32) [unless, > I suspect, it is set by MCI_DGV_PUT_SOURCE]; would you mind adding an > explicit test for that? > > + > > + /* test default video destination size */ > > + err = mciSendCommandW(wDeviceID, MCI_WHERE, > MCI_DGV_WHERE_DESTINATION, (DWORD_PTR)&parm); > > + ok(!err,"mciCommand where destination returned %s\n", dbg_mcierr(err)); > > + if (style & (WS_POPUP|WS_CHILD)) > > + rc = src_rc; > > + else > > + GetClientRect(video_window, &rc); > > + todo_wine_if(style & (WS_POPUP|WS_CHILD)) > > + ok(EqualRect(&parm.where.rc, &rc), "got %s, expected %s\n", > wine_dbgstr_rect(&parm.where.rc), wine_dbgstr_rect(&rc)); > And this should too. The test here is a bit awkward as such because the > destination and source rect are *always* equal, and separately, the > client rect will also be equal to the video size if the window is a > popup or child. > > + > > + /* test default video window size */ > > + rc = src_rc; > > + AdjustWindowRect(&rc, WS_OVERLAPPEDWINDOW, FALSE); /* sigh */ > > + OffsetRect(&rc, -rc.left, -rc.top); > > + rc.right = max(rc.right, GetSystemMetrics(SM_CXMIN)); > I gather "sigh" here means "Windows is broken", in the way spelled out > in 8/11, right? It wouldn't hurt to spell that out explicitly ;-) > _______________________________________________ > wine-gitlab mailing list -- wine-gitlab(a)winehq.org > To unsubscribe send an email to wine-gitlab-leave(a)winehq.org > ``` ``` > @@ -1573,6 +1575,37 @@ static void test_video_window(void) > todo_wine_if(i != 3) > ok(style == expected, "hwnd %p: got %#lx, expected %#lx\n", video_window, style, expected); > > + /* get source video size */ > + err = mciSendCommandW(wDeviceID, MCI_WHERE, MCI_DGV_WHERE_SOURCE, (DWORD_PTR)&parm); > + ok(!err,"mciCommand where source returned %s\n", dbg_mcierr(err)); > + src_rc = parm.where.rc; ``` > This should just be equal to the whole video rect (0,0)-(64,32) [unless, > I suspect, it is set by MCI_DGV_PUT_SOURCE]; would you mind adding an > explicit test for that? Not at all. Actually, video rect is (0,0)-(32,24). ``` > + > + /* test default video destination size */ > + err = mciSendCommandW(wDeviceID, MCI_WHERE, MCI_DGV_WHERE_DESTINATION, (DWORD_PTR)&parm); > + ok(!err,"mciCommand where destination returned %s\n", dbg_mcierr(err)); > + if (style & (WS_POPUP|WS_CHILD)) > + rc = src_rc; > + else > + GetClientRect(video_window, &rc); > + todo_wine_if(style & (WS_POPUP|WS_CHILD)) > + ok(EqualRect(&parm.where.rc, &rc), "got %s, expected %s\n", wine_dbgstr_rect(&parm.where.rc), wine_dbgstr_rect(&rc)); ``` > And this should too. The test here is a bit awkward as such because the > destination and source rect are *always* equal, and separately, the > client rect will also be equal to the video size if the window is a > popup or child. Minimum width of video window is SM_CXMIN. So, if the video width less than SM_CXMIN approximately, destination rect isn't identical to the source one. ``` > + > + /* test default video window size */ > + rc = src_rc; > + AdjustWindowRect(&rc, WS_OVERLAPPEDWINDOW, FALSE); /* sigh */ > + OffsetRect(&rc, -rc.left, -rc.top); > + rc.right = max(rc.right, GetSystemMetrics(SM_CXMIN)); ``` > I gather "sigh" here means "Windows is broken", in the way spelled out > in 8/11, right? It wouldn't hurt to spell that out explicitly ;-) Right. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/557#note_5586 [View Less]

2 1

Re: [PATCH v3 0/11] MR557: mciqtz32: Correct video window behavior by creating default window.
by Akihiro Sagawa (＠sgwaki) 03 Aug '22

03 Aug '22

On Tue Aug 2 05:14:13 2022 +0000, **** wrote: > Zebediah Figura (she/her) replied on the mailing list: > ``` > On 7/31/22 06:53, Akihiro Sagawa wrote: > > +static BOOL MCIQTZ_CreateWindow(WINE_MCIQTZ* wma, DWORD dwFlags, > LPMCI_DGV_OPEN_PARMSW lpParms) > > +{ > > + HWND parent = NULL; > > + DWORD style = (WS_OVERLAPPEDWINDOW | WS_CLIPCHILDREN) & ~WS_MAXIMIZEBOX; > > + LONG width, height; > > + RECT rc; > > + HRESULT hr; … [View More]

1 0

Re: [PATCH v3 0/11] MR557: mciqtz32: Correct video window behavior by creating default window.
by Akihiro Sagawa (＠sgwaki) 03 Aug '22

03 Aug '22

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 … [View More]

1 0

2025

2024

2023

2022

wine-gitlab