Re: [PATCH 0/1] MR2773: include: Add xamlom.idl file.

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about include/xamlom.idl:
> +{
> - HRESULT OnElementStateChanged(
> -
>    [in] InstanceHandle element,
> -
>    [in] VisualElementState elementState,
> -
>    [in] LPCWSTR context);
>
> +}
> -
>
> +[
> - object,
> - uuid(a593b11a-d17f-48bb-8f66-83910731c8a5),
> - pointer_default(unique)
>
> +]
> +interface IVisualTreeService : IUnknown
> +{
> - HRESULT AdviseVisualTreeChange(
> -
>    [in] IVisualTreeServiceCallback *pCallback);

I don't think we need a line break here. I also don't think we want to be using this Hungarian notation or camel case for parameter names.