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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7319#note_94455