Re: [PATCH 0/4] MR7319: comctl32: Beginning of SysLink IAccessible implementation.

[Connor McAdams (＠cmcadams)]

Connor McAdams (@cmcadams) commented about dlls/comctl32/syslink.c:
> #define SL_RIGHTMARGIN  (0)
>  #define SL_BOTTOMMARGIN (0)
>
> +static inline SYSLINK_ACC *impl_from_IAccessible(IAccessible *iface)
> +{
> - return CONTAINING_RECORD(iface, SYSLINK_ACC, IAccessible_iface);
>
> +}
> -
>
> +static HRESULT WINAPI Accessible_QueryInterface(IAccessible *iface, REFIID iid, void **ppv)
> +{
> - SYSLINK_ACC *This = impl_from_IAccessible(iface);
> - TRACE("(%p,%s,%p)\n", iface, debugstr_guid(iid), ppv);

Nitpicky, but a newline between `SYSLINK_ACC *This = impl_from_IAccessible(iface);` and the TRACE message seems like a good idea.