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

[Connor McAdams (＠cmcadams)]

Connor McAdams (@cmcadams) commented about include/xamlom.idl:
> -
>    [out] unsigned int* pPropertyCount,
> -
>    [out, size_is(, *pPropertyCount)] PropertyChainValue** ppPropertyValues);
> -
> - HRESULT SetProperty(
> -
>    [in] InstanceHandle instanceHandle,
> -
>    [in] InstanceHandle value,
> -
>    [in] unsigned int propertyIndex);
> -
> - HRESULT ClearProperty(
> -
>    [in] InstanceHandle instanceHandle,
> -
>    [in] unsigned int propertyIndex);
> -
> - HRESULT GetCollectionCount(
> -
>    [in] InstanceHandle instanceHandle,
> -
>    [out] unsigned int* pCollectionSize);
> -

The asterisk here should be right behind the method argument name, and not at the end of the type. I.e:
`unsigned int *pCollectionSize`, and then remove the camel case/Hungarian notation to make it `unsigned int *collection_size`. This should be the same for all other method arguments in the other interfaces, as said before.