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

Re: [PATCH v7 0/5] MR8438: cfgmgr32: Device Query API part 1: Add a basic implementation for DevGetObjects.
by Vibhav Pant (＠vibhavp) June 30, 2025

June 30, 2025

On Mon Jun 30 13:17:18 2025 +0000, Rémi Bernon wrote: > 77622615b657feedfb379f51346b4d05a2cbd0c4 still fails with: > ``` > cfgmgr32.c:561: Test succeeded inside todo block: got len 0 > cfgmgr32.c:562: Test succeeded inside todo block: got objects 0000000000000000 > cfgmgr32.c:566: Test succeeded inside todo block: got len 0 > cfgmgr32.c:567: Test succeeded inside todo block: got objects 0000000000000000 > ``` Apologies, I forgot to run final tests on this commit. Should be fixed now, thanks. Since the returned code in these two cases is `S_OK`, I have set both `len` and `objects` to dummy values to test whether they are correct set to zero/NULL respectively. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8438#note_108361

1 0

[PATCH v2 0/6] MR8358: user32: Preparatory cleanup of class name version logic.
by Rémi Bernon June 30, 2025

June 30, 2025

Always copying client class name to temporary buffer, to which version string also points to, in order to convert integral class names to string and unify the logic for versioning later on. -- v2: user32: Always copy class names to a temporary buffer. comctl32: Fix class name case in manifest. user32: Move get_class_version out of get_class_info. user32: Load class module in get_class_version. user32: Move user32_module default out of get_class_info. user32: Init class name outside of get_versioned_name. https://gitlab.winehq.org/wine/wine/-/merge_requests/8358

3 10

Re: [PATCH v3 0/5] MR8460: ntdll: Add support for multi-battery systems and energy-reporting batteries on Linux
by Vibhav Pant (＠vibhavp) June 30, 2025

June 30, 2025

It might be better to make this an enum, and add some documentation for how different values affect how `MaxCapacity` and `RemainingCapacity` get calculated. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8460#note_108347

1 0

Re: [PATCH v5 0/5] MR8438: cfgmgr32: Device Query API part 1: Add a basic implementation for DevGetObjects.
by Rémi Bernon June 30, 2025

June 30, 2025

77622615b657feedfb379f51346b4d05a2cbd0c4 still fails with: ``` cfgmgr32.c:561: Test succeeded inside todo block: got len 0 cfgmgr32.c:562: Test succeeded inside todo block: got objects 0000000000000000 cfgmgr32.c:566: Test succeeded inside todo block: got len 0 cfgmgr32.c:567: Test succeeded inside todo block: got objects 0000000000000000 ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8438#note_108344

1 0

Re: [PATCH v3 0/5] MR8460: ntdll: Add support for multi-battery systems and energy-reporting batteries on Linux
by Tomas-Mendes-ai (＠Tomas-Mendes-ai) June 30, 2025

June 30, 2025

On Mon Jun 30 15:44:52 2025 +0000, Vibhav Pant wrote: > Shouldn't this be `!bat->present`? Hi Vibhav, Fear it should, yes. Thank you for catching that. Anything else you'd like to add before I edit the commit and rerun the pipeline? Thank you. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8460#note_108343

1 0

Re: [PATCH v3 0/5] MR8460: ntdll: Add support for multi-battery systems and energy-reporting batteries on Linux
by Tomas-Mendes-ai (＠Tomas-Mendes-ai) June 30, 2025

June 30, 2025

On Mon Jun 30 10:40:44 2025 +0000, Rémi Bernon wrote: > Are you an AI yourself? Has this been written with an AI? Hi Rémi. No on both accounts. Just an unfortunate automatic suggestion made by github when LLMs were not really a _thing_ yet. That being said, if somethings seems off about the code it's probably because I haven't dabbed in C since college. Any critiques or suggestions are much appreciated. Thanks. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8460#note_108342

1 0

Re: [PATCH v3 0/5] MR8460: ntdll: Add support for multi-battery systems and energy-reporting batteries on Linux
by Vibhav Pant (＠vibhavp) June 30, 2025

June 30, 2025

Shouldn't this be `!bat->present`? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8460#note_108339

1 0

Re: [PATCH v5 0/0] MR8378: urlmon: Add PersistentZoneIdentifier COM object - closed
by Mike Kozelkov (＠augenzi) June 30, 2025

June 30, 2025

This merge request was closed by Mike Kozelkov. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8378

1 0

Re: [PATCH v3 0/5] MR8460: ntdll: Add support for multi-battery systems and energy-reporting batteries on Linux
by Rémi Bernon June 30, 2025

June 30, 2025

Are you an AI yourself? Has this been written with an AI? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8460#note_108336

1 0

Re: [PATCH v2 0/1] MR8455: kernel32: Update len in the absolute path case.
by Huw Davies (＠huw) June 30, 2025

June 30, 2025

As pointed out in !8457, this previously failed to deal with the case when the file didn't exist. There are many ways of fixing this; here's one. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8455#note_108334

1 0