Two small listview corrections.

Rein Klazes rklazes at xs4all.nl
Tue Nov 18 05:15:23 CST 2003


Hi,

two bits as discussed on wine-devel.

Changelog:
	dlls/comctl32	: listview.c
	The LVN_GETDISPINFO notify message should point to the same
	iSubItem as in the call to LISTVIEW_GetItemT, valid or not. Fix
	a badly placed brace.

Rein.
-- 
Rein Klazes
rklazes at xs4all.nl
-------------- next part --------------
--- wine/dlls/comctl32/listview.c	2003-11-18 11:42:20.000000000 +0100
+++ mywine/dlls/comctl32/listview.c	2003-11-18 12:00:55.000000000 +0100
@@ -5084,7 +5084,8 @@
     {
 	SUBITEM_INFO *lpSubItem = LISTVIEW_GetSubItemPtr(hdpaSubItems, isubitem);
         pItemHdr = lpSubItem ? &lpSubItem->hdr : &callbackHdr;
-        if( !lpSubItem) {
+        if (!lpSubItem)
+        {
             WARN(" iSubItem invalid (%08x), ignored.\n", isubitem);
             isubitem = 0;
         }
@@ -5121,7 +5122,7 @@
     if (dispInfo.item.mask != 0)
     {
 	dispInfo.item.iItem = lpLVItem->iItem;
-	dispInfo.item.iSubItem = isubitem;
+	dispInfo.item.iSubItem = lpLVItem->iSubItem; /* yes: the original subitem */
 	dispInfo.item.lParam = lpItem->lParam;
 	notify_dispinfoT(infoPtr, LVN_GETDISPINFOW, &dispInfo, isW);
 	TRACE("   getdispinfo(2):item=%s\n", debuglvitem_t(&dispInfo.item, isW));


More information about the wine-patches mailing list