Michael Stefaniuc : comctl32/tests: Get rid of variables holding empty strings.
Michael Stefaniuc
mstefani at winehq.org
Tue Oct 22 17:43:22 CDT 2019
Uhm, this patch introduces test failures. I had assumed the Marvin email
is enough to reject this with "Test failures".
I doubt this patch is the real culprit but I didn't have time to
properly debug the issue.
On 10/22/19 11:57 PM, Alexandre Julliard wrote:
> Module: wine
> Branch: master
> Commit: f8979fee1fa2e6aeed5a5026465dffa680c0848f
> URL: https://source.winehq.org/git/wine.git/?a=commit;h=f8979fee1fa2e6aeed5a5026465dffa680c0848f
>
> Author: Michael Stefaniuc <mstefani at winehq.org>
> Date: Tue Oct 22 01:24:32 2019 +0200
>
> comctl32/tests: Get rid of variables holding empty strings.
>
> Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
> Signed-off-by: Alexandre Julliard <julliard at winehq.org>
>
> ---
> diff --git a/dlls/comctl32/tests/pager.c b/dlls/comctl32/tests/pager.c
> index 0e2c25b08a..30005c88fa 100644
> --- a/dlls/comctl32/tests/pager.c
> +++ b/dlls/comctl32/tests/pager.c
> @@ -132,17 +130,17 @@ static const struct notify_test_send test_dont_convert_send_data[] =
>
> static const struct notify_test_receive test_convert_receive_data[] =
> {
> - {empty_w, sizeof(empty_w), ARRAY_SIZE(buffer), NULL, test_a, sizeof(test_a), -1, test_w, ARRAY_SIZE(buffer)},
> - {empty_w, sizeof(empty_w), ARRAY_SIZE(buffer), test_a, NULL, 0, -1, test_w, ARRAY_SIZE(buffer)},
> - {NULL, sizeof(empty_w), ARRAY_SIZE(buffer), test_a, NULL, 0, -1, NULL, ARRAY_SIZE(buffer)},
> - {empty_w, sizeof(empty_w), ARRAY_SIZE(buffer), large_a, NULL, 0, -1, large_truncated_65_w, ARRAY_SIZE(buffer)},
> - {empty_w, sizeof(empty_w), ARRAY_SIZE(buffer), empty_a, 0, 0, 1, empty_w, 1},
> + {L"", sizeof(L""), ARRAY_SIZE(buffer), NULL, test_a, sizeof(test_a), -1, test_w, ARRAY_SIZE(buffer)},
> + {L"", sizeof(L""), ARRAY_SIZE(buffer), test_a, NULL, 0, -1, test_w, ARRAY_SIZE(buffer)},
> + {NULL, sizeof(L""), ARRAY_SIZE(buffer), test_a, NULL, 0, -1, NULL, ARRAY_SIZE(buffer)},
> + {L"", sizeof(L""), ARRAY_SIZE(buffer), large_a, NULL, 0, -1, large_truncated_65_w, ARRAY_SIZE(buffer)}, ^^^ changing that back to empty_w makes the test failures go away...
bye
michael
More information about the wine-devel
mailing list