Re: [PATCH 0/8] MR7402: comctl32: Implement IAccessible methods.

[Connor McAdams (＠cmcadams)]

Connor McAdams (@cmcadams) commented about dlls/comctl32/syslink.c:
> static HRESULT WINAPI Accessible_get_accName(IAccessible *iface, VARIANT childid, BSTR *name)
>  {
> - FIXME("%p\n", iface);
> - return E_NOTIMPL;
> - SYSLINK_ACC *This = impl_from_IAccessible(iface);
> - HRESULT hr;
> - DOC_ITEM* item;
> - BSTR result;
> -
> - TRACE("%p, %s\n", iface, debugstr_variant(&childid));
> -

Adding a `NULL` check for name and clearing it to `NULL` in case of failure would probably be useful here to avoid potentially returning an uninitialized variable upon failure.