[PATCH v2 2/2] user32/listbox: Use is_item_selected in GetSelCount and GetSelItems
Huw Davies
huw at codeweavers.com
Mon Feb 25 04:31:01 CST 2019
From: Gabriel Ivăncescu <gabrielopcode at gmail.com>
Signed-off-by: Gabriel Ivăncescu <gabrielopcode at gmail.com>
Signed-off-by: Huw Davies <huw at codeweavers.com>
---
dlls/user32/listbox.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/dlls/user32/listbox.c b/dlls/user32/listbox.c
index 0be48673ca..826ce01cc4 100644
--- a/dlls/user32/listbox.c
+++ b/dlls/user32/listbox.c
@@ -1066,13 +1066,12 @@ static INT LISTBOX_FindString( LB_DESCR *descr, INT start, LPCWSTR str, BOOL exa
static LRESULT LISTBOX_GetSelCount( const LB_DESCR *descr )
{
INT i, count;
- const LB_ITEMDATA *item = descr->items;
if (!(descr->style & LBS_MULTIPLESEL) ||
(descr->style & LBS_NOSEL))
return LB_ERR;
- for (i = count = 0; i < descr->nb_items; i++, item++)
- if (item->selected) count++;
+ for (i = count = 0; i < descr->nb_items; i++)
+ if (is_item_selected(descr, i)) count++;
return count;
}
@@ -1083,11 +1082,10 @@ static LRESULT LISTBOX_GetSelCount( const LB_DESCR *descr )
static LRESULT LISTBOX_GetSelItems( const LB_DESCR *descr, INT max, LPINT array )
{
INT i, count;
- const LB_ITEMDATA *item = descr->items;
if (!(descr->style & LBS_MULTIPLESEL)) return LB_ERR;
- for (i = count = 0; (i < descr->nb_items) && (count < max); i++, item++)
- if (item->selected) array[count++] = i;
+ for (i = count = 0; (i < descr->nb_items) && (count < max); i++)
+ if (is_item_selected(descr, i)) array[count++] = i;
return count;
}
--
2.18.0
More information about the wine-devel
mailing list