Re: [PATCH 0/4] MR7386: d3dx9: Add support for saving surfaces to targa files.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/surface.c:
> -
>        buffer = NULL;
> -
>        hr = D3DXSaveSurfaceToFileInMemory(&buffer, test_iff[j], surface, iff_tests[i].palette, NULL);
> -
>        todo_wine_if(iff_tests[i].iff_expected[j].todo_hr) ok(hr == iff_tests[i].iff_expected[j].hr, "Unexpected hr %#lx.\n", hr);
> -
>        if (SUCCEEDED(hr))
> -
>        {
> -
>            D3DXIMAGE_INFO info = { 0 };
> -
> -
>            hr = D3DXGetImageInfoFromFileInMemory(ID3DXBuffer_GetBufferPointer(buffer), ID3DXBuffer_GetBufferSize(buffer), &info);
> -
>            ok(hr == D3D_OK, "Unexpected hr %#lx.\n", hr);
> -
>            todo_wine_if(iff_tests[i].iff_expected[j].todo_format) ok(info.Format == iff_tests[i].iff_expected[j].format,
> -
>                    "Unexpected image format %u (%s), expected %u (%s).\n", info.Format, debug_d3dformat(info.Format),
> -
>                    iff_tests[i].iff_expected[j].format, debug_d3dformat(iff_tests[i].iff_expected[j].format));
> -
>            ID3DXBuffer_Release(buffer);
> -
>        }
> -
>        winetest_pop_context();
> -
>    }

There are some very long lines here, in general let's try to stay under 100-120 columns (no specific hard limit, more of a rough guideline).