Re: [PATCH v2 0/7] MR5666: d3dx9: Introduce d3dx_resource structures for use in code sharing.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/surface.c:
> -
>    else if ((src_image_size >= 10) && !strncmp(src_image, "#?RADIANCE", 10))
> -
>        FIXME("File type HDR is not supported yet.\n");
> -
>    else if ((src_image_size >= 2) && (!strncmp(src_image, "PF", 2) || !strncmp(src_image, "Pf", 2)))
> -
>        FIXME("File type PFM is not supported yet.\n");
> -
>    goto exit;
> - }
> -
> - hr = IWICBitmapDecoder_GetContainerFormat(bitmap_decoder, &container_format);
> - if (FAILED(hr))
> -
>    goto exit;
> -
> - info->ImageFileFormat = wic_container_guid_to_d3dx_file_format(&container_format);
> - if (is_dib && info->ImageFileFormat == D3DXIFF_BMP)
> -
>    info->ImageFileFormat = D3DXIFF_DIB;
> - else if (info->ImageFileFormat == D3DXIFF_FORCE_DWORD)
> - {

Style comment, usually we prefer to have `{` `}` on both branches of an `if` when you need them for one anyway.