[PATCH] comctl32/tests: Avoid an unneeded lstrlenW() call.

[Francois Gouget]

Note that buffer_w is an array and thus cannot be NULL.

Signed-off-by: Francois Gouget <fgouget(a)free.fr>
---
 dlls/comctl32/tests/button.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/comctl32/tests/button.c b/dlls/comctl32/tests/button.c
index dd05a98d2dd..14071b2ecd0 100644
--- a/dlls/comctl32/tests/button.c
+++ b/dlls/comctl32/tests/button.c
@@ -1166,7 +1166,7 @@ static void test_note(void)
             size = ARRAY_SIZE(buffer_w);
             ret = SendMessageA(hwnd, BCM_GETNOTE, (WPARAM)&size, (LPARAM)buffer_w);
             ok(ret, "Expect BCM_GETNOTE return true\n");
-            ok(lstrlenW(buffer_w) == 0, "Expect note length 0\n");
+            ok(!*buffer_w, "Expect note length 0\n");
             ok(size == ARRAY_SIZE(buffer_w), "Got: %d\n", size);
             error = GetLastError();
             ok(error == NO_ERROR, "Expect last error: 0x%08x, got: 0x%08x\n", NO_ERROR, error);
-- 
2.20.1

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=58105

Your paranoid android.


=== wvistau64_fr (32 bit report) ===

comctl32:
button.c:825: Test failed: BM_SETSTATE/TRUE on a button: the msg sequence is not complete: expected 0000 - actual 000f