April 2025 - wine-gitlab

[PATCH 0/1] MR7918: windows.ui/tests: Remove tests for exact color values.
by Nikolay Sivov (＠nsivov) 26 Apr '25

26 Apr '25

The tests are failing on Windows 11. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7918

2 1

[PATCH 0/1] MR7906: vwin32.vxd: Add support for VxDCall() 0x0010 and 0x002a services
by Pali Rohár (＠pali) 26 Apr '25

26 Apr '25

VxDCall service id 0x0010 provides int24 dispatch functionality and VxDCall service id 0x002a provides int41 dispatch functionality. All of these services together with existing service id 0x0029 (for int31/dpmi functionality) have same API, first VxDCall() argument is value for EAX register and second argument is value for ECX register. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7906

3 2

[PATCH 0/2] MR7915: shell32: Fix use-after-free at FolderItemVerbs creation (ASan).
by Nikolay Sivov (＠nsivov) 26 Apr '25

26 Apr '25

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7915

2 2

[PATCH 0/1] MR7914: gdiplus: Don't trace old values in GdipSetMatrixElements.
by Esme Povirk (＠madewokherd) 26 Apr '25

26 Apr '25

These are about to be overwritten, so they're not really meaningful at all. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7914

2 1

Re: [PATCH v24 0/1] MR7843: cmd: Implement tab completion for command line entry.
by Joe Souza (＠JoeS209) 26 Apr '25

26 Apr '25

On Sat Apr 26 17:30:57 2025 +0000, Joe Souza wrote: > I wasn't aware of WCMD_parameter. That is a good suggestion if it works > for what's needed here. FindFirstFileEx is an optimization for remote/network file systems, such that a round trip across the network is not performed for unwanted directory entries. Since Wine seems to support only file systems that are mounted by the host operating system and does not appear to have any direct knowledge of remote-mounted file systems, I can skip using the Ex function here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7843#note_101956

1 0

Re: [PATCH v24 0/1] MR7843: cmd: Implement tab completion for command line entry.
by Joe Souza (＠JoeS209) 26 Apr '25

26 Apr '25

On Sat Apr 26 17:30:01 2025 +0000, Joe Souza wrote: > In the latest changes I have already split out much code from > WCMD_ReadConsole into subfunctions. You're saying it's still too long? > I suppose that I could split out the code for the individual case > statements into separate handler functions, but the code to handle Tab > is the bulk of that function. I wasn't aware of WCMD_parameter. That is a good suggestion if it works for what's needed here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7843#note_101955

1 0

Re: [PATCH v24 0/1] MR7843: cmd: Implement tab completion for command line entry.
by Joe Souza (＠JoeS209) 26 Apr '25

26 Apr '25

On Sat Apr 26 15:12:31 2025 +0000, eric pouech wrote: > thanks > a couple of high level comments: > * I don't think having a separate source file just for \~400 LoC is > worth it > * WCMD_ReadConsole is way too big as a function and need to be split in > smaller pieces. > * you should also be using the command line parsing functions instead of > rewriting them (eg WCMD_parameter to loop against the various parameters) > from quick testing: > * when looping around files sometimes filename is inserted in quotes, > while it shouldn't. Native seems to insert quotes only when filename > contains specific characters (spaces...), but to also remove them if > next file doesn't contain any... > * using "cd c:\\Program Files"\<tab\>\<backspace\>\\\<tab\> doesn't > behave as native... native circles in the directory content, current MR > only adds a double quote > * got a crash when playing with MR (perhaps one of the buffer overflow > stated above, perhaps not) > * when the width of the console is smaller than the full line input (eg > spread at least on two lines), circling gives some strange result > (cursor and display when from bottom of screen buffer back to top, will > previous content not shifted) > likely the current appoach of just having a handle to findfirst/findnext > will not fit all the (long term) goals. at some point, we want to > support shift-tab which circles backwards in the matching files. and I > fail to see what using FindFirstFileEx brings as value, as you filter > against directory when needed (I'm not asking for supporting shift-tab > right now, but rather to put in place a structure were it wil fit simply > when we implement it) > support for cd/rmdir/ and friends should be added in a second patch > I didn't go further in the review given the amount a remarks and > problems stated above. I'll restart when they are solved > (will look in // to the ESC issue raised) In the latest changes I have already split out much code from WCMD_ReadConsole into subfunctions. You're saying it's still too long? I suppose that I could split out the code for the individual case statements into separate handler functions, but the code to handle Tab is the bulk of that function. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7843#note_101954

1 0

Re: [PATCH v24 0/1] MR7843: cmd: Implement tab completion for command line entry.
by Joe Souza (＠JoeS209) 26 Apr '25

26 Apr '25

On Sat Apr 26 15:12:31 2025 +0000, eric pouech wrote: > that's not sufficient, you have also to compare that inputbuffer up to > current pos hasn't changed since previous call > (eg directory with za, za2 and zb zb2 files... circle twice from "za", > replace "za" by "zb" and circle with tab...) I missed that case. I tested C:\win<tab> and C:\pro<tab> but resulting paths are different lengths. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7843#note_101953

1 0

Re: [PATCH v24 0/1] MR7843: cmd: Implement tab completion for command line entry.
by Joe Souza (＠JoeS209) 26 Apr '25

26 Apr '25

On Sat Apr 26 15:12:31 2025 +0000, eric pouech wrote: > looks to be that curPos can be \> \*numRead, hence reading garbage in > that case Line 312 terminates the buffer at *numRead, line 314 sets len to *numRead, and loop at 320 stops when !(curPos < len). Worst case is if curPos == len then we are checking the null terminator. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7843#note_101952

1 0

[PATCH v2 0/1] MR7888: wbemprox: Implement Win32_CacheMemory table.
by Paul Gofman (＠gofman) 26 Apr '25

26 Apr '25

-- v2: wbemprox: Implement Win32_CacheMemory table. https://gitlab.winehq.org/wine/wine/-/merge_requests/7888

3 3

2025

2024

2023

2022

wine-gitlab April 2025