Re: [PATCH v14 0/4] MR6315: plugplay: Rework how PnP notifications are sent over RPC to support DBT_DEVTYP_HANDLE events.

[Elizabeth Figura (＠zfigura)]

Elizabeth Figura (@zfigura) commented about include/wine/plugplay.idl:
> import "wtypes.idl";
>
> +typedef enum _broadcast_device_type
> +{
> - /* DBT_DEVTYP_DEVICEINTERFACE */
> - BROADCAST_DEVICE_TYPE_DEVICEINTERFACE = 5,
> - /* DBT_DEVTYP_HANDLE */
> - BROADCAST_DEVICE_TYPE_HANDLE = 6
>
> +} broadcast_device_type;
> -
>
> +typedef struct _device_broadcast_header
> +{
> - DWORD reserved;
>
> +} device_broadcast_header;
> -
>
> +typedef struct _device_broadcast_deviceinterface

I would get rid of the typedef as well. Generally I think 'struct X' is better by virtue of being more explicit.