[PATCH] comctl32/tests: Avoid a couple of unneeded lstrlenA() calls.

Francois Gouget fgouget at free.fr
Mon Dec 9 02:43:01 CST 2019


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

Signed-off-by: Francois Gouget <fgouget at free.fr>
---
 dlls/comctl32/tests/updown.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/comctl32/tests/updown.c b/dlls/comctl32/tests/updown.c
index e7ceeffd4cd..d0fff562499 100644
--- a/dlls/comctl32/tests/updown.c
+++ b/dlls/comctl32/tests/updown.c
@@ -791,7 +791,7 @@ static void test_updown_create(void)
     flush_sequences(sequences, NUM_MSG_SEQUENCES);
 
     GetWindowTextA(g_edit, text, MAX_PATH);
-    ok(lstrlenA(text) == 0, "Expected empty string\n");
+    ok(!*text, "Expected empty string\n");
     ok_sequence(sequences, EDIT_SEQ_INDEX, get_edit_text_seq, "get edit text", FALSE);
 
     DestroyWindow(updown);
@@ -865,7 +865,7 @@ static void test_UDS_SETBUDDYINT(void)
     ok(style & UDS_SETBUDDYINT, "Expected UDS_SETBUDDY to be set\n");
     SendMessageA(updown, UDM_SETPOS, 0, 20);
     GetWindowTextA(g_edit, text, ARRAY_SIZE(text));
-    ok(lstrlenA(text) == 0, "Expected empty string\n");
+    ok(!*text, "Expected empty string\n");
     DestroyWindow(updown);
 
     /* creating with UDS_SETBUDDYINT */
-- 
2.20.1




More information about the wine-devel mailing list