[PATCH v4 1/5] server: Added support for tracking registered device notifications

Alexandre Julliard julliard at winehq.org
Wed Jun 12 10:13:32 CDT 2019


Micah N Gorrell <mgorrell at codeweavers.com> writes:

> diff --git a/server/user.h b/server/user.h
> index eb1b7ce1e4..282cb63339 100644
> --- a/server/user.h
> +++ b/server/user.h
> @@ -36,7 +36,8 @@ enum user_object
>  {
>      USER_WINDOW = 1,
>      USER_HOOK,
> -    USER_CLIENT  /* arbitrary client handle */
> +    USER_CLIENT, /* arbitrary client handle */
> +    USER_DEVNOTIFY

I don't think these should be user handles, especially since they don't
work across processes. But if they are, you'd need some mechanism to
clean them up when the client dies.

To be honest I'm not convinced these should be in the server at
all. Probably they could just as well be registered directly with the
service that sends the notification.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list