March 2024 - wine-gitlab

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

21 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. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5369

4 3

Re: [PATCH v2 0/7] MR5261: ntdll: Only allocate debug info in critical sections with RTL_CRITICAL_SECTION_FLAG_FORCE_DEBUG_INFO.
by Paul Gofman (＠gofman) 21 Mar '24

21 Mar '24

I am reproducing the test failure, will take a look. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5261#note_65687

1 0

Re: [PATCH v2 0/7] MR5261: ntdll: Only allocate debug info in critical sections with RTL_CRITICAL_SECTION_FLAG_FORCE_DEBUG_INFO.
by Rémi Bernon (＠rbernon) 21 Mar '24

21 Mar '24

I have no idea how, but somehow 8b3944e1341baaf693927c8b758851d2dbba725a caused some mf:mf tests to start failing (IMFSession_Start returning S_OK instead of MF_E_SHUTDOWN). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5261#note_65686

1 0

[PATCH 0/1] MR5375: server: Avoid removing thread twice from its desktop thread list.
by Rémi Bernon (＠rbernon) 21 Mar '24

21 Mar '24

When a process is made a system process, `release_thread_desktop( thread, 0 )` is called and we remove the process threads from their desktop thread list, as well as decrementing the desktop user count. Later, when the process gets killed on shutdown, `release_thread_desktop( thread, 1 )` is called, and we remove the thread list node again. This is 1) wrong, we only want to decrement the desktop user count in this case, and 2) causes an invalid write and corrupts the list the second time we remove the entry. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5375

2 1

[PATCH v27 0/9] MR3103: server: Create and use a desktop shared mapping for GetCursorPos.
by Rémi Bernon (＠rbernon) 21 Mar '24

21 Mar '24

First part of Proton shared memory series. The full branch can be seen at https://gitlab.winehq.org/rbernon/wine/-/commits/mr/shared-memories. -- v27: win32u: Use the desktop shared data for GetCursorPos. server: Move the last cursor time to the desktop session object. server: Move the cursor position to the desktop session object. win32u: Open desktop shared objects from session mapping. server: Return the desktop object info in get_thread_desktop. server: Allocate shared session object for desktops. win32u: Open the global session shared mapping. include: Add ReadNoFence64 inline helpers. server: Create a global session shared mapping. https://gitlab.winehq.org/wine/wine/-/merge_requests/3103

4 27

[PATCH v2 0/4] MR5329: ddraw: Support multiple devices per ddraw object.
by Paul Gofman (＠gofman) 21 Mar '24

21 Mar '24

Fixes Sim City 3000 Building Architect. -- v2: ddraw/tests: Add tests for multiple devices. ddraw: Support multiple devices per ddraw object. https://gitlab.winehq.org/wine/wine/-/merge_requests/5329

5 14

[PATCH v4 0/1] MR5357: d3dcompiler: Add D3DX11CreateThreadPump stub.
by Vijay Kiran Kamuju (＠infyquest) 21 Mar '24

21 Mar '24

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56176 -- v4: d3dcompiler: Add D3DX11CreateThreadPump stub. https://gitlab.winehq.org/wine/wine/-/merge_requests/5357

3 2

[PATCH v3 0/1] MR5236: ntdll: Implement NtdllDefWindowProc_A().
by Aida Jonikienė (＠DodoGTA) 21 Mar '24

21 Mar '24

I've tried implementing this before but importing user32 from ntdll isn't a reliable thing (so I recently rewrote it using the ntdll equivalents of LoadLibrary() and GetProcAddress()) Note: This is a weird way to implement this function (Windows implements this in ntdll instead) but that would probably require a user32/ntdll rewrite (so this is the next best tning to get the application working) -- v3: ntdll: Implement NtdllDefWindowProc_A(). https://gitlab.winehq.org/wine/wine/-/merge_requests/5236

5 11

[PATCH v3 0/8] MR5346: Support generic xstate (part 3).
by Paul Gofman (＠gofman) 21 Mar '24

21 Mar '24

-- v3: ntdll/tests: Add more tests for xstate. ntdll, wineboot: Support more xstate features. ntdll: Mind generic xstate masks in server context conversion. https://gitlab.winehq.org/wine/wine/-/merge_requests/5346

2 9

[PATCH 0/5] MR5374: Winedbg for minidump.
by eric pouech (＠epo) 21 Mar '24

21 Mar '24

This series: - add a couple of commands to winedbg to ease scripting (mainly reload a minidump from parser, run a debuggee with arguments...). - a couple of quick fixes (updating CPU info displayed on reload) and detecting reloading a minidump for a different machine. WOW64 support in minidump (dbghelp & winedbg) will require quite some work. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5374

2 5

2025

2024

2023

2022

wine-gitlab March 2024