Commit 7decba4e authored by Dmitry Timoshkov's avatar Dmitry Timoshkov Committed by Alexandre Julliard

comctl32/listbox: Fix order of items passed in WM_COMPAREITEM data.

parent e7bcc119
...@@ -798,7 +798,7 @@ static INT LISTBOX_FindStringPos( LB_DESCR *descr, LPCWSTR str, BOOL exact ) ...@@ -798,7 +798,7 @@ static INT LISTBOX_FindStringPos( LB_DESCR *descr, LPCWSTR str, BOOL exact )
{ {
index = (min + max) / 2; index = (min + max) / 2;
if (HAS_STRINGS(descr)) if (HAS_STRINGS(descr))
res = LISTBOX_lstrcmpiW( descr->locale, str, descr->items[index].str); res = LISTBOX_lstrcmpiW( descr->locale, descr->items[index].str, str );
else else
{ {
COMPAREITEMSTRUCT cis; COMPAREITEMSTRUCT cis;
...@@ -809,15 +809,15 @@ static INT LISTBOX_FindStringPos( LB_DESCR *descr, LPCWSTR str, BOOL exact ) ...@@ -809,15 +809,15 @@ static INT LISTBOX_FindStringPos( LB_DESCR *descr, LPCWSTR str, BOOL exact )
cis.hwndItem = descr->self; cis.hwndItem = descr->self;
/* note that some application (MetaStock) expects the second item /* note that some application (MetaStock) expects the second item
* to be in the listbox */ * to be in the listbox */
cis.itemID1 = -1; cis.itemID1 = index;
cis.itemData1 = (ULONG_PTR)str; cis.itemData1 = descr->items[index].data;
cis.itemID2 = index; cis.itemID2 = -1;
cis.itemData2 = descr->items[index].data; cis.itemData2 = (ULONG_PTR)str;
cis.dwLocaleId = descr->locale; cis.dwLocaleId = descr->locale;
res = SendMessageW( descr->owner, WM_COMPAREITEM, id, (LPARAM)&cis ); res = SendMessageW( descr->owner, WM_COMPAREITEM, id, (LPARAM)&cis );
} }
if (!res) return index; if (!res) return index;
if (res < 0) max = index; if (res > 0) max = index;
else min = index + 1; else min = index + 1;
} }
return exact ? -1 : max; return exact ? -1 : max;
......
...@@ -326,9 +326,7 @@ static LRESULT WINAPI main_window_proc(HWND hwnd, UINT msg, WPARAM wParam, LPARA ...@@ -326,9 +326,7 @@ static LRESULT WINAPI main_window_proc(HWND hwnd, UINT msg, WPARAM wParam, LPARA
ok(wParam == cis->CtlID, "expected %#x, got %#lx\n", cis->CtlID, wParam); ok(wParam == cis->CtlID, "expected %#x, got %#lx\n", cis->CtlID, wParam);
ok(cis->hwndItem == ctrl, "expected %p, got %p\n", ctrl, cis->hwndItem); ok(cis->hwndItem == ctrl, "expected %p, got %p\n", ctrl, cis->hwndItem);
todo_wine
ok((int)cis->itemID1 >= 0, "expected >= 0, got %d\n", cis->itemID1); ok((int)cis->itemID1 >= 0, "expected >= 0, got %d\n", cis->itemID1);
todo_wine
ok((int)cis->itemID2 == -1, "expected -1, got %d\n", cis->itemID2); ok((int)cis->itemID2 == -1, "expected -1, got %d\n", cis->itemID2);
if (cis->CtlType == ODT_LISTBOX) if (cis->CtlType == ODT_LISTBOX)
...@@ -2030,10 +2028,8 @@ static void test_WM_MEASUREITEM(void) ...@@ -2030,10 +2028,8 @@ static void test_WM_MEASUREITEM(void)
ret = SendMessageA(listbox, LB_ADDSTRING, 0, (LPARAM)"item 0"); ret = SendMessageA(listbox, LB_ADDSTRING, 0, (LPARAM)"item 0");
ok(ret == 0, "expected 0, got %ld\n", ret); ok(ret == 0, "expected 0, got %ld\n", ret);
ret = SendMessageA(listbox, LB_ADDSTRING, 0, (LPARAM)"item 1"); ret = SendMessageA(listbox, LB_ADDSTRING, 0, (LPARAM)"item 1");
todo_wine
ok(ret == 1, "expected 1, got %ld\n", ret); ok(ret == 1, "expected 1, got %ld\n", ret);
ret = SendMessageA(listbox, LB_ADDSTRING, 0, (LPARAM)"item 2"); ret = SendMessageA(listbox, LB_ADDSTRING, 0, (LPARAM)"item 2");
todo_wine
ok(ret == 2, "expected 2, got %ld\n", ret); ok(ret == 2, "expected 2, got %ld\n", ret);
ok_sequence(sequences, PARENT_SEQ_INDEX, lb_addstring_sort_parent_seq, "LB_ADDSTRING (LBS_SORT)", TRUE); ok_sequence(sequences, PARENT_SEQ_INDEX, lb_addstring_sort_parent_seq, "LB_ADDSTRING (LBS_SORT)", TRUE);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment