user32: LB_SETITEMDATA should return 1 on success, not 0 [try2]

[Peter Verthez]

Second try after comment from Alexandre (avoid creating constant that doesn't 
exist on Windows)

---
 ChangeLog                   |    9 +++++++++
 dlls/user32/listbox.c       |    3 ++-
 dlls/user32/tests/listbox.c |   22 ++++++++++++++++++++++
 3 files changed, 33 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 8fe4354..4d75d23 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2007-08-14  Peter Verthez  <Peter.Verthez at advalvas.be>
+
+	* dlls/user32/tests/listbox.c:
+	Added test for LB_SETITEMDATA..
+
+	* dlls/user32/listbox.c, include/winuser.h:
+	user32: StraceNt utility shows that LB_SETITEMDATA should return 1
+	on success, not 0 (this is undocumented).
+
 2007-08-10  Paul Vriens <paul.vriens.wine at gmail.com>
 
 	* dlls/snmpapi/tests/util.c:
diff --git a/dlls/user32/listbox.c b/dlls/user32/listbox.c
index 166d176..84eee66 100644
--- a/dlls/user32/listbox.c
+++ b/dlls/user32/listbox.c
@@ -2677,7 +2677,8 @@ static LRESULT WINAPI ListBoxWndProc_common( HWND hwnd, UINT msg,
             return LB_ERR;
         }
         descr->items[wParam].data = lParam;
-        return LB_OKAY;
+        /* Undocumented: Command returns 1 on windows, not LB_OKAY (0) */
+        return TRUE;
 
     case LB_GETCOUNT16:
     case LB_GETCOUNT:
diff --git a/dlls/user32/tests/listbox.c b/dlls/user32/tests/listbox.c
index 125bd9d..e3cece0 100644
--- a/dlls/user32/tests/listbox.c
+++ b/dlls/user32/tests/listbox.c
@@ -512,6 +512,27 @@ static void test_itemfrompoint(void)
     DestroyWindow( hList );
 }
 
+static void test_listbox_item_data(void)
+{
+    HWND hList;
+    int r, id;
+
+    hList = CreateWindow( "ListBox", "list test", 0, 
+                          1, 1, 600, 100, NULL, NULL, NULL, NULL );
+    ok( hList != NULL, "failed to create listbox\n");
+
+    id = SendMessage( hList, LB_ADDSTRING, 0, (LPARAM) "hi");
+    ok( id == 0, "item id wrong\n");
+
+    r = SendMessage( hList, LB_SETITEMDATA, 0, MAKELPARAM( 20, 0 ));
+    ok(r == TRUE, "LB_SETITEMDATA returned %d instead of TRUE\n", r);
+
+    r = SendMessage( hList, LB_GETITEMDATA, 0, 0);
+    ok( r == 20, "get item data failed\n");
+
+    DestroyWindow( hList );
+}
+
 START_TEST(listbox)
 {
   const struct listbox_test SS =
@@ -587,4 +608,5 @@ START_TEST(listbox)
   test_selection();
   test_listbox_height();
   test_itemfrompoint();
+  test_listbox_item_data();
 }
-- 
1.5.2.2