August 2024 - Wine-gitlab - list.winehq.org

Re: [PATCH v2 0/3] MR6407: include: Updated a few gdiplus types. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/1] MR6412: winecoreaudio: Set the synth volume to the greater of the left and right channels. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/1] MR6415: mscoree: Update Wine Mono to 9.3.0. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/1] MR6416: secur32: Ensure unixlib function tables and enum stay in sync. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/1] MR6414: secur32: Simplify the cred_enabled_protocols logic slightly. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/5] MR6411: Cmd engine rewrite (part XXIII) - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v32 0/4] MR6315: plugplay: Rework how PnP notifications are sent over RPC to support DBT_DEVTYP_HANDLE events.
by Rémi Bernon 30 Aug '24

30 Aug '24

On Fri Aug 30 18:00:21 2024 +0000, Vibhav Pant wrote: > > Maybe we can avoid sending the paths every time by moving the > filtering to plugplay, adapting the interface a bit for this. There > could be additional methods to register paths / get plugplay handles in > return, and later use these handle as device handles. > Perhaps, but wouldn't this make both plugplay service, and the > notification code sechost/service.c a lot more stateful as a result? Re: > device handles, if you're referring to the `dbch_handle` field in > `DEV_BROADCAST_HANDLE`, we already get that from the user using > `RegisterDeviceNotification`, and the current code just uses them for > sending notifications. I don't think adding custom structs is justified at this point, especially as it requires rewriting large parts of the code in multiple modules at the same time. > Perhaps, but wouldn't this make both plugplay service, and the notification code sechost/service.c a lot more stateful as a result? Re: device handles, if you're referring to the `dbch_handle` field in `DEV_BROADCAST_HANDLE`, we already get that from the user using `RegisterDeviceNotification`, and the current code just uses them for sending notifications. Fwiw I'm fine keeping the paths, and using the `reserved` field to describe them. I was just considering additional improvements but this specifically doesn't need to be done now. However, passing process-private handles around to other processes and later getting them back looks wrong. What happens if you register for notification on a handle then close it right away? Does the notification keep firing with the old handle value? It seem surprising and it would be a nice thing to test for instance. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6315#note_80567

1 0

Re: [PATCH v2 0/1] MR6413: msi: Bump version to 5.0. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/2] MR6410: msiexec: Don't remove quotes from properties passed on the command line. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0

Re: [PATCH v2 0/1] MR6403: winemac.drv: Call CGWindowListCreateImageFromArray through a dlsym-obtained pointer. - approved
by Alexandre Julliard (＠julliard) 30 Aug '24

30 Aug '24

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

1 0