Re: [PATCH v2 0/3] MR6704: d3dx9: Some changes to pixel format handling on the WIC image decode path.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/surface.c:
> static HRESULT d3dx_image_wic_frame_decode(struct d3dx_image *image,
>          IWICImagingFactory *wic_factory, IWICBitmapFrameDecode *bitmap_frame)
>  {
> - const GUID *dst_pixel_format = d3dformat_to_wic_guid(image->format);

Not really relevant to this MR but as a general consideration: I think we usually name those "mapping" functions the other way around, like in this case `wic_guid_from_d3dformat()`. I didn't really notice this earlier but it slightly confused me here, not sure why.

Again, nothing to change here and now, but maybe something to keep in mind when you happen to touch any of those functions...