February 2025 - Wine-GitLab

Re: [PATCH v2 0/1] MR7346: msvcrt/tests: Add missing parameter pmode to _open call (ASan). - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

Re: [PATCH v3 0/2] MR7343: Add some d2d effect stubs. - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

Re: [PATCH v4 0/14] MR7329: windowscodecs/tests: Add some query tests with the Unknown reader. - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

Re: [PATCH v11 0/4] MR7241: winewayland: Basic input method support. - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

Re: [PATCH v2 0/2] MR7338: Add FlushProcessWriteBuffers() tests - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

Re: [PATCH v2 0/5] MR7195: setuapi: Implement a few cfgmgr functions. - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

Re: [PATCH v3 0/1] MR7210: setupapi: Use wide character string literals. - approved
by Alexandre Julliard (＠julliard) Feb. 17, 2025

Feb. 17, 2025

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

1 0

[PATCH v2 0/5] MR7347: ntdll: Avoid attach even as dependency if loaded with `DONT_RESOLVE_DLL_REFERENCES`
by Jinoh Kang (＠iamahuman) Feb. 17, 2025

Feb. 17, 2025

This cleans up `LDR_DONT_RESOLVE_REFS` checks in preparation for !7. -- v2: ntdlll: Remove redundant LDR_DONT_RESOLVE_REFS checks before calling process_attach(). ntdll: Skip DLL initialization and ldr notification entirely if DONT_RESOLVE_DLL_REFERENCES is set. kernel32/tests: Test for unexpected loader notification for import dependency loaded with DONT_RESOLVE_DLL_REFERENCES. kernel32/tests: Test for unexpected LDR_PROCESS_ATTACHED flag in import dependency loaded with DONT_RESOLVE_DLL_REFERENCES. kernel32/tests: Add basic tests for internal flags of modules loaded with DONT_RESOLVE_DLL_REFERENCES. https://gitlab.winehq.org/wine/wine/-/merge_requests/7347

3 6

[PATCH v13 0/1] MR7322: programs/cmd: Implement ability to abort lengthy directory operations via Ctrl-C.
by Joe Souza (＠JoeS209) Feb. 17, 2025

Feb. 17, 2025

Allow the user to press Ctrl-C to abort lengthy DIR (or DIR /p, etc.) operations. -- v13: programs/cmd: Implement ability to abort lengthy directory operations via Ctrl-C. https://gitlab.winehq.org/wine/wine/-/merge_requests/7322

2 2

Re: [PATCH v12 0/1] MR7322: programs/cmd: Implement ability to abort lengthy directory operations via Ctrl-C.
by Joe Souza (＠JoeS209) Feb. 17, 2025

Feb. 17, 2025

On Mon Feb 17 19:22:08 2025 +0000, eric pouech wrote: > thanks for taking the time to update your patch > unfortunately, it fails on wine regression test > the attached patch should solve this > [patch.ctrlc](/uploads/84ae84d1c5cf3d3dfc19904c673bc906/patch.ctrlc) > also, any reason you changed the ctrl event for and automatic reset? > actually, I was puzzled with the previous versions with a manual reset > (my first guess would have been to implement it with an automatic reset) > I tried things like '(dir /s \\\* || echo aaaaa) && dir \*' (and halting > the first dir command with ctrl-c) on native: > * does break the first dir command, > * executes the RHS of || (as the dir command failed because of the > ctrl-c), > * prints the 'aaaaa' string > * executes the RHS of && (as last command of LHS, eg echo, did succeed), > * outputs just the directory header (but nothing else) for the second dir > which fits better with a manual reset (it could be implemented in native > with something else, like a global variable set in the event handler > -requires atomic ops-) Thank you for the patch. I will try to get that integrated sometime today. I'll be traveling for the next few days and will have limited ability during that time to address further concerns. Automatic vs. manual reset were based on my thinking about whether the function WCMD_ctrlc_status would be called more than once per command iteration. If called only once then ownership of the status passes to the caller and thus automatic reset is cleaner. However, thinking further about this, it probably makes sense to switch back to manual reset, in case future callers (i.e. perhaps for different internal commands, etc.) might want to call it more than once per iteration. That, and the fact that there is a manual reset anyway before command launch in the loop toward the end of wmain suggests that a manual reset would be better, so I will switch back when I submit the changes containing your patch. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7322#note_94891

1 0