Re: [PATCH] user32: Add an exception handler to GetWindowText.

[Rémi Bernon]

On 10/13/21 11:41 AM, Dmitry Timoshkov wrote:
> Signed-off-by: Dmitry Timoshkov
>
> dlls/user32/tests/win.c | 29 ++++++++++++++++----
>   dlls/user32/win.c       | 59 ++++++++++++++++++++++++++++++-----------
>   2 files changed, 67 insertions(+), 21 deletions(-)
>
> diff --git a/dlls/user32/tests/win.c b/dlls/user32/tests/win.c
> index 6b13d93be14..2679ea0813f 100644
> --- a/dlls/user32/tests/win.c
> +++ b/dlls/user32/tests/win.c
> @@ -7923,17 +7923,36 @@ static void test_gettext(void)
>       if (0)
>       {
>       r = SendMessageA( hwnd, WM_GETTEXT, 0x10, 0x1000);
> - ok( r == 0, "settext should return zero\n");
> -
>
> ok(r == 0, "WM_GETTEXT should return zero (%ld)\n", r);
>
> r = SendMessageA( hwnd, WM_GETTEXT, 0x10000, 0);
> - ok( r == 0, "settext should return zero (%ld)\n", r);
> -
>
> ok(r == 0, "WM_GETTEXT should return zero (%ld)\n", r);
>
> r = SendMessageA( hwnd, WM_GETTEXT, 0xff000000, 0x1000);
> - ok( r == 0, "settext should return zero (%ld)\n", r);
> -
>
> ok(r == 0, "WM_GETTEXT should return zero (%ld)\n", r);
>
> r = SendMessageA( hwnd, WM_GETTEXT, 0x1000, 0xff000000);
> - ok( r == 0, "settext should return zero (%ld)\n", r);
> - }
> - ok(r == 0, "WM_GETTEXT should return zero (%ld)\n", r);
> - }

FWIW everything else in this test function (and as far as I could see 
most user32, or at least recently added, code) uses space-in-paren 
style, if you're fixing the spaces I think it should be the other way 
around ;)