Re: [PATCH v10 0/4] MR6304: comctl32/edit: Fix incorrect size for format rect.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/user32/tests/edit.c:
> -
>    {ES_MULTILINE | WS_VISIBLE, 0, {1, 1, 1000, 1000}, 1},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {0, 0, 0, 0}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {0, 0, 10, 10}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {1, 1, 10, 10}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {1, 1, 10, 250}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {1, 1, 250, 10}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {1, 1, 10, 1000}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {1, 1, 1000, 10}, 0},
> -
>    {ES_MULTILINE | WS_VISIBLE, WS_EX_CLIENTEDGE, {1, 1, 1000, 1000}, 1}
> - };
> -
> - for (int i = 0; i < ARRAY_SIZE(tests); i++)
> - {
> -
>    edit = create_editcontrol(tests[i].style, tests[i].style_ex);
> -
> -
>    SendMessageA(edit, EM_GETRECT, 0, (LPARAM)&expected_rect);

Let's rename expected_rect to old_rect. It seems more appropriate this way because the new rectangle doesn't always equal to the old rectangle.