commit c5552a694b07e2dea0b788717bb696b8b765f9bf
author Peter Åstrand <astrand@cendio.se> Tue Mar 07 14:07:49 2006 +0100
committer Alexandre Julliard <julliard@winehq.org> Wed Mar 08 11:32:46 2006 +0100
tree c404a392af35466bb72d5b698559f2a3a4ad7fe0
parent 1d491aedba0b28f984f921568a84667745b0745e

comctl32: LISTVIEW_InsertItemT should accept iItem < 0, when using sorted lists.

dlls/comctl32/listview.c

diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c
index 9210e8d..cab119a 100644
--- a/dlls/comctl32/listview.c
+++ b/dlls/comctl32/listview.c
@@ -6211,7 +6211,7 @@
     if (infoPtr->dwStyle & LVS_OWNERDATA) return infoPtr->nItemCount++;
 
     /* make sure it's an item, and not a subitem; cannot insert a subitem */
-    if (!lpLVItem || lpLVItem->iItem < 0 || lpLVItem->iSubItem) return -1;
+    if (!lpLVItem || lpLVItem->iSubItem) return -1;
 
     if (!is_assignable_item(lpLVItem, infoPtr->dwStyle)) return -1;
 
@@ -6224,6 +6224,8 @@
     is_sorted = (infoPtr->dwStyle & (LVS_SORTASCENDING | LVS_SORTDESCENDING)) &&
 	        !(infoPtr->dwStyle & LVS_OWNERDRAWFIXED) && (LPSTR_TEXTCALLBACKW != lpLVItem->pszText);
 
+    if (lpLVItem->iItem < 0 && !is_sorted) return -1;
+
     nItem = is_sorted ? infoPtr->nItemCount : min(lpLVItem->iItem, infoPtr->nItemCount);
     TRACE(" inserting at %d, sorted=%d, count=%d, iItem=%d\n", nItem, is_sorted, infoPtr->nItemCount, lpLVItem->iItem);
     nItem = DPA_InsertPtr( infoPtr->hdpaItems, nItem, hdpaSubItems );