Re: [PATCH v5 0/1] MR8367: shell32: Try to lookup icon, with fallback on failure

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/shell32/iconcache.c:
> - sii->iIcon = -IDI_SHELL_FILE;
> - sii->iIcon = id;
> lstrcatW(sii->szPath, L"\shell32.dll");
> - if (flags)
> -
>    FIXME("flags 0x%x not implemented\n", flags);
> -
>
> if (flags & ~SHGSI_ICON)
> -
>    FIXME("unhandled flags 0x%x\n", flags);
>
> sii->hIcon = NULL;
> - if (flags & SHGSI_ICON)
> -
>    sii->hIcon = LoadIconW(GetModuleHandleW(sii->szPath), MAKEINTRESOURCEW(sii->iIcon));
> - if (flags & SHGSI_ICON) {
> -
>    sii->hIcon = LoadIconW(GetModuleHandleW(sii->szPath), MAKEINTRESOURCEW(StockIconIdToResourceId(id)));
> -
> -
>    // many icons do not exist yet in wine, map to the question mark

Use /**/ for comments.