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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2773#note_34801