March 2024 - Wine-gitlab - list.winehq.org

Re: [PATCH v3 0/1] MR5361: user32: Add LoadKeyboardLayoutEx stub.
by Rémi Bernon 22 Mar '24

22 Mar '24

Rémi Bernon (@rbernon) commented about dlls/user32/tests/input.c: > } > } > > +static void test_keyboard_layoutex(void) ```suggestion:-0+0 static void test_LoadKeyboardLayoutEx(void) ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5361#note_65789

1 0

Re: [PATCH v3 0/1] MR4878: advpack: Ignore lines that begin with '@' in (Un)RegisterOCXs sections.
by Alexandre Julliard (＠julliard) 22 Mar '24

22 Mar '24

Alexandre Julliard (@julliard) commented about dlls/advpack/tests/install.c: > "Signature=\"$Chicago$\"\n" > "AdvancedINF=2.5\n" > "[DefaultInstall]\n" > - "CheckAdminRights=1\n"; > + "CheckAdminRights=1\n" > + "[OcxInstall]\n" > + "RegisterOCXs=OCXsToRegister\n" > + "[OCXsToRegister]\n" > + "@foobar\n" > + "@foobaz\n"; That's not a very convincing test. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4878#note_65786

1 0

[PATCH v2 0/1] MR5382: gitlab: Output collapsible section markers in test stage script.
by Jinoh Kang (＠iamahuman) 22 Mar '24

22 Mar '24

See pipeline "test-linux-*" job output for preview. -- v2: gitlab: Output collapsible section markers in test stage script. https://gitlab.winehq.org/wine/wine/-/merge_requests/5382

2 1

[PATCH v8 0/3] MR734: vkd3d-shader/dxil: SamplePosition and SampleCount.
by Conor McCarthy (＠cmccarthy) 22 Mar '24

22 Mar '24

-- v8: vkd3d-shader/dxil: Implement DX intrinsic RenderTargetGetSampleCount. vkd3d-shader/dxil: Implement DX intrinsics Texture2DMSGetSamplePosition and RenderTargetGetSamplePosition. tests/hlsl: Add tests for GetSamplePosition() and GetSampleCount(). https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/734

2 3

[PATCH v7 0/3] MR734: vkd3d-shader/dxil: SamplePosition and SampleCount.
by Conor McCarthy (＠cmccarthy) 22 Mar '24

22 Mar '24

-- v7: vkd3d-shader/dxil: Implement DX intrinsic RenderTargetGetSampleCount. vkd3d-shader/dxil: Implement DX intrinsics Texture2DMSGetSamplePosition and RenderTargetGetSamplePosition. tests/hlsl: Add tests for GetSamplePosition() and GetSampleCount(). https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/734

2 3

[PATCH 0/1] MR5382: gitlab: Output collapsible section markers in test stage script.
by Jinoh Kang (＠iamahuman) 22 Mar '24

22 Mar '24

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/5382

2 2

[PATCH v3 0/2] MR5238: krnl386, mmsystem: Simulate a hardware interrupt in MMSYSTDRV_Callback3216.
by Alex Henrie (＠alexhenrie) 22 Mar '24

22 Mar '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56260 -- v3: mmsystem: Simulate a hardware interrupt in MMSYSTDRV_Callback3216. krnl386: Add a function to simulate a hardware interrupt. https://gitlab.winehq.org/wine/wine/-/merge_requests/5238

2 2

Re: [PATCH v4 0/1] MR1060: include: Remove interfaces already define in msxml6.idl
by Zebediah Figura (＠zfigura) 22 Mar '24

22 Mar '24

On Sat Mar 16 23:37:29 2024 +0000, Alistair Leslie-Hughes wrote: > I currently don't have one. > Through wine-staging patches for FreeThreadedXMLHTTP60 interface rely on > this change. Is fixing the headers undesirable, then? Is it necessary in some way to implement FreeThreadedXMLHTTP60, or can that be done differently? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1060#note_65760

1 0

[PATCH v3 0/1] MR5369: Draft: winegstreamer: Refactor wg_format.
by Ziqing Hui (＠zhui) 22 Mar '24

22 Mar '24

Since we are adding more and more media formats to wg_format, the current wg_format struct is getting ugly. Everytime we add a video format, we need to duplicate width, height, fps. Everytime we add a audio format, we need to duplicate channels, rate. So it would be better if we could share width, height, fps fields between different video formats, also share channels and rate fields between different audio formats. What makes me found the current wg_format is not in a good shape is when I was writting code for Proton, I found that I need to write some code like this if want to get width/height/fps from a wg_format: ``` static bool get_video_info_from_wg_format(struct wg_format *format, int32_t *width, int32_t *height, uint32_t *fps_n, uint32_t *fps_d) { switch (format->major_type) { case WG_MAJOR_TYPE_VIDEO: *width = format->u.video.width; *height = format->u.video.height; *fps_n = format->u.video.fps_n; *fps_d = format->u.video.fps_d; return true; case WG_MAJOR_TYPE_VIDEO_CINEPAK: *width = format->u.video_cinepak.width; *height = format->u.video_cinepak.height; *fps_n = format->u.video_cinepak.fps_n; *fps_d = format->u.video_cinepak.fps_d; return true; case WG_MAJOR_TYPE_VIDEO_H264: *width = format->u.video_h264.width; *height = format->u.video_h264.height; *fps_n = format->u.video_h264.fps_n; *fps_d = format->u.video_h264.fps_d; return true; case WG_MAJOR_TYPE_VIDEO_WMV: *width = format->u.video_wmv.width; *height = format->u.video_wmv.height; *fps_n = format->u.video_wmv.fps_n; *fps_d = format->u.video_wmv.fps_d; return true; case WG_MAJOR_TYPE_VIDEO_INDEO: *width = format->u.video_indeo.width; *height = format->u.video_indeo.height; *fps_n = format->u.video_indeo.fps_n; *fps_d = format->u.video_indeo.fps_d; return true; case WG_MAJOR_TYPE_VIDEO_MPEG1: *width = format->u.video_mpeg1.width; *height = format->u.video_mpeg1.height; *fps_n = format->u.video_mpeg1.fps_n; *fps_d = format->u.video_mpeg1.fps_d; return true; default: GST_ERROR("Type %d is not a video format.\n", format->major_type); return false; } } ``` Apparently, the code above is ugly. By refactoring wg_format, we can avoid code like this. This patch is a draft now, it only contains unixlib.h code. Zeb, I'd like to confirm that if this refactoring would be acceptable for you. If do, I'll continue finishing the remaining parts. -- v3: winegstreamer: Refactor wg_format. https://gitlab.winehq.org/wine/wine/-/merge_requests/5369

4 4

[PATCH v6 0/4] MR734: vkd3d-shader/dxil: SamplePosition and SampleCount.
by Conor McCarthy (＠cmccarthy) 22 Mar '24

22 Mar '24

-- v6: tests/shader-runner: Check for D3D11 MSAA support. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/734

2 4