April 2025 - Wine-GitLab - list.winehq.org

[PATCH 0/1] MR7664: ntdll: Add copies of two XDG env vars to the environment, and use those in...
by Louis Lenders (＠xe) April 27, 2025

April 27, 2025

ntdll: Add copies of two XDG env vars to the environment, and use those in winemenubuilder and shell32. To solve https://bugs.winehq.org/show_bug.cgi?id=49987 and https://bugs.winehq.org/show_bug.cgi?id=58023 we have to remove these two XDG variables from the environment. This is a proposal that adds copies with a different name and after this they can be removed. If this is not the right way to go I guess I will hear something? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7664

2 3

[PATCH v2 0/1] MR7652: ntdll: Treat XDG_DATA_DIRS as special env variable.
by Louis Lenders (＠xe) April 27, 2025

April 27, 2025

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=49987 GTK apps seem to use this to find path to schemas, like Meld: https://download.gnome.org/binaries/win32/meld/3.22/Meld-3.22.2-mingw.msi which doesn't start at all without this patch. More info in https://bugs.winehq.org/show_bug.cgi?id=49987. -- v2: ntdll: Treat XDG_DATA_DIRS as special env variable. https://gitlab.winehq.org/wine/wine/-/merge_requests/7652

3 6

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

April 26, 2025

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 April 26, 2025

April 26, 2025

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

2 2

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

April 26, 2025

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) April 26, 2025

April 26, 2025

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) April 26, 2025

April 26, 2025

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) April 26, 2025

April 26, 2025

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) April 26, 2025

April 26, 2025

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) April 26, 2025

April 26, 2025

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