Re: [PATCH 0/5] MR9558: d2d1: Add some new effect stubs

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/d2d1/tests/d2d1.c:
> - release_test_context(&ctx);
>
> +}
> -
>
> +static void test_effect_saturation(BOOL d3d11)
> +{
> - static const struct effect_property properties[] =
> - {
> -
>    { L"Saturation", D2D1_SATURATION_PROP_SATURATION, D2D1_PROPERTY_TYPE_FLOAT },
> - };
> - float f;
> - struct d2d1_test_context ctx;
> - ID2D1DeviceContext *context;
> - unsigned int count, i;
> - ID2D1Effect *effect;
> - WCHAR name[64];
> - HRESULT hr;

Here, and in other functions too, this is normally sorted by length, so 'float f;' would go at the end. See formatting of existing tests.