comctl32[6/6]: toolbar: fix the return code of TB_ADDBITMAP
Mikołaj Zalewski
mikolaj at zalewski.pl
Wed Sep 20 10:52:24 CDT 2006
-------------- next part --------------
diff --git a/dlls/comctl32/tests/toolbar.c b/dlls/comctl32/tests/toolbar.c
index d1277d8..096bfbe 100644
--- a/dlls/comctl32/tests/toolbar.c
+++ b/dlls/comctl32/tests/toolbar.c
@@ -173,22 +173,22 @@ void test_add_bitmap()
CHECK_IMAGELIST(13, 16, 15);
/* adding the same bitmap will simply return the index of the already loaded block */
ret = SendMessageA(hToolbar, TB_ADDBITMAP, 8, (LPARAM)&bmp128);
- todo_wine ok(ret == 0, "TB_ADDBITMAP - unexpected return %d\n", ret);
- CHECK_IMAGELIST_TODO_COUNT(13, 16, 15);
+ ok(ret == 0, "TB_ADDBITMAP - unexpected return %d\n", ret);
+ CHECK_IMAGELIST(13, 16, 15);
ret = SendMessageA(hToolbar, TB_ADDBITMAP, 5, (LPARAM)&bmp80);
- todo_wine ok(ret == 8, "TB_ADDBITMAP - unexpected return %d\n", ret);
- CHECK_IMAGELIST_TODO_COUNT(13, 16, 15);
+ ok(ret == 8, "TB_ADDBITMAP - unexpected return %d\n", ret);
+ CHECK_IMAGELIST(13, 16, 15);
/* even if we increase the wParam */
ret = SendMessageA(hToolbar, TB_ADDBITMAP, 55, (LPARAM)&bmp80);
- todo_wine ok(ret == 8, "TB_ADDBITMAP - unexpected return %d\n", ret);
- CHECK_IMAGELIST_TODO_COUNT(13, 16, 15);
+ ok(ret == 8, "TB_ADDBITMAP - unexpected return %d\n", ret);
+ CHECK_IMAGELIST(13, 16, 15);
/* when the wParam is smaller than the bitmaps count but non-zero, all the bitmaps will be added*/
rebuild_toolbar(&hToolbar);
ok(SendMessageA(hToolbar, TB_ADDBITMAP, 3, (LPARAM)&bmp128) == 0, "TB_ADDBITMAP - unexpected return\n");
CHECK_IMAGELIST(8, 16, 15);
ret = SendMessageA(hToolbar, TB_ADDBITMAP, 5, (LPARAM)&bmp80);
- todo_wine ok(ret == 3, "TB_ADDBITMAP - unexpected return %d\n", ret);
+ ok(ret == 3, "TB_ADDBITMAP - unexpected return %d\n", ret);
/* the returned value is misleading - id 8 is the id of the first icon from bmp80 */
CHECK_IMAGELIST(13, 16, 15);
@@ -198,7 +198,7 @@ void test_add_bitmap()
ok(ret == 0, "TB_ADDBITMAP - unexpected return %d\n", ret);
CHECK_IMAGELIST(8, 16, 15);
ret = SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&bmp80);
- todo_wine ok(ret == -143, "TB_ADDBITMAP - unexpected return %d\n", ret);
+ ok(ret == -143, "TB_ADDBITMAP - unexpected return %d\n", ret);
CHECK_IMAGELIST(13, 16, 15);
/* for zero only one bitmap will be added */
@@ -230,14 +230,14 @@ void test_add_bitmap()
ok(SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&bmp128) == 0, "TB_ADDBITMAP - unexpected return\n");
CHECK_IMAGELIST(6, 20, 20);
ret = SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&bmp80);
- todo_wine ok(ret == 1, "TB_ADDBITMAP - unexpected return %d\n", ret);
+ ok(ret == 1, "TB_ADDBITMAP - unexpected return %d\n", ret);
CHECK_IMAGELIST(10, 20, 20);
/* the icons can be resized - an UpdateWindow is needed as this probably happens during WM_PAINT */
ok(SendMessageA(hToolbar, TB_SETBITMAPSIZE, 0, MAKELONG(8, 8)) == TRUE, "TB_SETBITMAPSIZE failed\n");
UpdateWindow(hToolbar);
CHECK_IMAGELIST_TODO_COUNT_SIZE(26, 8, 8);
/* loading a standard bitmaps automatically resizes the icons */
- todo_wine ok(SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&stdsmall) == 2, "TB_ADDBITMAP - unexpected return\n");
+ ok(SendMessageA(hToolbar, TB_ADDBITMAP, 1, (LPARAM)&stdsmall) == 2, "TB_ADDBITMAP - unexpected return\n");
UpdateWindow(hToolbar);
CHECK_IMAGELIST_TODO_COUNT_SIZE(28, 16, 15);
diff --git a/dlls/comctl32/toolbar.c b/dlls/comctl32/toolbar.c
index 4222d15..f7c5aae 100644
--- a/dlls/comctl32/toolbar.c
+++ b/dlls/comctl32/toolbar.c
@@ -2580,6 +2580,7 @@ TOOLBAR_AddBitmap (HWND hwnd, WPARAM wPa
LPTBADDBITMAP lpAddBmp = (LPTBADDBITMAP)lParam;
TBITMAP_INFO info;
INT nIndex = 0, nCount;
+ INT iSumButtons, i;
HBITMAP hbmLoad;
HIMAGELIST himlDef;
@@ -2652,6 +2653,16 @@ TOOLBAR_AddBitmap (HWND hwnd, WPARAM wPa
info.nID = lpAddBmp->nID;
TRACE("adding %d bitmaps!\n", info.nButtons);
}
+
+ /* check if the bitmap is already loaded and compute iSumButtons */
+ iSumButtons = 0;
+ for (i = 0; i < infoPtr->nNumBitmapInfos; i++)
+ {
+ if (infoPtr->bitmaps[i].hInst == info.hInst &&
+ infoPtr->bitmaps[i].nID == info.nID)
+ return iSumButtons;
+ iSumButtons += infoPtr->bitmaps[i].nButtons;
+ }
if (!infoPtr->cimlDef) {
/* create new default image list */
@@ -2733,7 +2744,7 @@ TOOLBAR_AddBitmap (HWND hwnd, WPARAM wPa
InvalidateRect(hwnd, NULL, TRUE);
- return nIndex;
+ return iSumButtons;
}
--
1.4.1
More information about the wine-patches
mailing list