Re: [PATCH 0/2] MR8555: windows.devices.enumeration: Reimplement DeviceInformationStatics::FindAllAsync and DeviceWatcher using the device query API.

[Vibhav Pant (＠vibhavp)]

On Wed Jul 30 08:13:21 2025 +0000, Rémi Bernon wrote:
> This doesn't look very useful to merge both cases after all, what about:
> ```suggestion:-16+0
>         case DevQueryStateClosed:
>             EnterCriticalSection( &watcher->cs );
>             watcher->status = DeviceWatcherStatus_Stopped;
>             watcher->query = NULL;
>             LeaveCriticalSection( &watcher->cs );
>             typed_event_handlers_notify( &watcher->stopped_handlers,
> (IInspectable *)iface, NULL );
>             IWeakReference_Release( weak );
>             break;
>         case DevQueryStateAborted:
>             EnterCriticalSection( &watcher->cs );
>             watcher->status = DeviceWatcherStatus_Aborted;
>             DevCloseObjectQuery( watcher->query );
>             watcher->query = NULL;
>             LeaveCriticalSection( &watcher->cs );
>             IWeakReference_Release( weak );
>             break;
> ```
> Yeah, there isn't much common code between them. Thanks.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8555#note_111476