=?UTF-8?Q?Gabriel=20Iv=C4=83ncescu=20?=: comctl32/listbox: Move the item data removal into a separate function.
Alexandre Julliard
julliard at winehq.org
Wed Feb 20 16:37:00 CST 2019
Module: wine
Branch: master
Commit: e98d4b4502ba75e8e206984e5d819aee0af3d670
URL: https://source.winehq.org/git/wine.git/?a=commit;h=e98d4b4502ba75e8e206984e5d819aee0af3d670
Author: Gabriel Ivăncescu <gabrielopcode at gmail.com>
Date: Wed Feb 20 12:04:59 2019 +0000
comctl32/listbox: Move the item data removal into a separate function.
Signed-off-by: Gabriel Ivăncescu <gabrielopcode at gmail.com>
Signed-off-by: Huw Davies <huw at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/comctl32/listbox.c | 19 ++++++++++---------
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/dlls/comctl32/listbox.c b/dlls/comctl32/listbox.c
index 44e3b1c..e89d257 100644
--- a/dlls/comctl32/listbox.c
+++ b/dlls/comctl32/listbox.c
@@ -200,6 +200,15 @@ static void insert_item_data(LB_DESCR *descr, UINT index, WCHAR *str, ULONG_PTR
item->selected = FALSE;
}
+static void remove_item_data(LB_DESCR *descr, UINT index)
+{
+ LB_ITEMDATA *item;
+
+ item = descr->items + index;
+ if (index < descr->nb_items)
+ memmove(item, item + 1, (descr->nb_items - index) * sizeof(LB_ITEMDATA));
+}
+
/***********************************************************************
* LISTBOX_GetCurrentPageSize
*
@@ -1699,8 +1708,6 @@ static void LISTBOX_DeleteItem( LB_DESCR *descr, INT index )
*/
static LRESULT LISTBOX_RemoveItem( LB_DESCR *descr, INT index )
{
- LB_ITEMDATA *item;
-
if ((index < 0) || (index >= descr->nb_items)) return LB_ERR;
/* We need to invalidate the original rect instead of the updated one. */
@@ -1713,15 +1720,9 @@ static LRESULT LISTBOX_RemoveItem( LB_DESCR *descr, INT index )
}
descr->nb_items--;
LISTBOX_DeleteItem( descr, index );
+ remove_item_data(descr, index);
- /* Remove the item */
-
- item = &descr->items[index];
- if (index < descr->nb_items)
- RtlMoveMemory( item, item + 1,
- (descr->nb_items - index) * sizeof(LB_ITEMDATA) );
if (descr->anchor_item == descr->nb_items) descr->anchor_item--;
-
resize_storage(descr, descr->nb_items);
/* Repaint the items */
More information about the wine-cvs
mailing list