Re: [PATCH 0/2] MR2489: include: Add Windows.System.DispatcherQueue declaration.

[Mohamad Al-Jaf (＠maljaf)]

Mohamad Al-Jaf (@maljaf) commented about include/windows.system.idl:
> -
>    contract(Windows.Foundation.UniversalApiContract, 5.0),
> -
>    uuid(dfa2dc9c-1a2d-4917-98f2-939af1d6e0c8)
> - ]
> - delegate HRESULT DispatcherQueueHandler();
> -
> - [
> -
>    contract(Windows.Foundation.UniversalApiContract, 5.0),
> -
>    exclusiveto(Windows.System.DispatcherQueue),
> -
>    uuid(603e88e4-a338-4ffe-a457-a5cfb9ceb899)
> - ]
> - interface IDispatcherQueue : IInspectable
> - {
> -
>    HRESULT CreateTimer([out, retval] Windows.System.DispatcherQueueTimer **result);
> -
>    [overload("TryEnqueue")] HRESULT TryEnqueue([in] Windows.System.DispatcherQueueHandler *callback, [out, retval] boolean *result);
> -
>    [overload("TryEnqueue")] HRESULT TryEnqueueWithPriority([in] Windows.System.DispatcherQueuePriority priority, [in] Windows.System.DispatcherQueueHandler *callback, [out, retval] boolean *result);
> -
>    [eventadd] HRESULT ShutdownStarting([in] Windows.Foundation.TypedEventHandler<Windows.System.DispatcherQueue *, Windows.System.DispatcherQueueShutdownStartingEventArgs *> *handler, [out, retval] EventRegistrationToken *token);

Some of these lines are a bit long, like these ones. The parameters are usually split like this:

```suggestion:-1+0
        [overload("TryEnqueue")] HRESULT TryEnqueueWithPriority([in] Windows.System.DispatcherQueuePriority priority, [in] Windows.System.DispatcherQueueHandler *callback,
                                                                [out, retval] boolean *result);
        [eventadd] HRESULT ShutdownStarting([in] Windows.Foundation.TypedEventHandler<Windows.System.DispatcherQueue *, Windows.System.DispatcherQueueShutdownStartingEventArgs *> *handler,
                                            [out, retval] EventRegistrationToken *token);
```