On Fri, Mar 24, 2000 at 07:42:44PM +0100, Marcus Meissner wrote:
> On Fri, Mar 24, 2000 at 05:17:45PM +0000, Huw D M Davies wrote:
> > --- controls/listbox.c 2000/02/26 19:13:12 1.37
> > +++ controls/listbox.c 2000/03/24 17:12:53
> > @@ -1269,7 +1269,7 @@
> > if (oldsel != -1) descr->items[oldsel].selected = FALSE;
> > if (index != -1) descr->items[index].selected = TRUE;
> > descr->selected_item = index;
> > - if (oldsel != -1) LISTBOX_RepaintItem( wnd, descr, oldsel, 0 );
> > + if (oldsel != -1) LISTBOX_RepaintItem( wnd, descr, oldsel, ODA_SELECT );
> > if (index != -1) LISTBOX_RepaintItem( wnd, descr, index, ODA_SELECT );
>
> Really? Should we mark ODA_SELECT the old, to be unselected one?
Well the docs say:
ODA_SELECT: the selection status has changed. The itemState member
[of DRAWITEMSTRUCT] should be checked to determine the new selection
state.
So this looks correct to me.
Huw.
--
Dr. Huw D M Davies | Clarendon Laboratory
[EMAIL PROTECTED] | Parks Road
Tel: +44 1865 272390 | Oxford OX1 3PU
Fax: +44 1865 272400 | UK
