Re: [PATCH v11 0/8] MR6892: propsys: Implement IPropertyDescription for several known system properties.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/propsys/propsys_desc.c:
> -
>    *out = iface;
> -
>    IUnknown_AddRef( iface );
> -
>    return S_OK;
> - }
> -
> - FIXME( "%s not implemented\n", debugstr_guid( iid ) );
> - return E_NOINTERFACE;
>
> +}
> -
>
> +static ULONG WINAPI propdesc_AddRef( IPropertyDescription *iface )
> +{
> - struct property_description *impl;
> - TRACE( "(%p)\n", iface );
> - impl = impl_from_IPropertyDescription( iface );
> - return InterlockedIncrement( &impl->ref );
>
> +}

Generally used style is to have impl_from_* as first line. Also please use better name than "impl".