From 90f1b587c46665fd66ac2328110b4588fef5bcfb Mon Sep 17 00:00:00 2001 From: kojima Date: Tue, 2 Nov 1999 21:28:08 +0000 Subject: [PATCH] fixed bug in repainting empty popup buttons --- WINGs/wpopupbutton.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/WINGs/wpopupbutton.c b/WINGs/wpopupbutton.c index 8fd2fcd1..b3db223a 100644 --- a/WINGs/wpopupbutton.c +++ b/WINGs/wpopupbutton.c @@ -180,7 +180,7 @@ WMRemovePopUpButtonItem(WMPopUpButton *bPtr, int index) if (index < bPtr->selectedItemIndex) bPtr->selectedItemIndex--; else if (index == bPtr->selectedItemIndex) { - /* reselect first item if the removed item is the + /* reselect first item if the removed item is the * selected one */ bPtr->selectedItemIndex = 0; if (bPtr->view->flags.mapped) @@ -301,6 +301,9 @@ WMGetPopUpButtonItem(WMPopUpButton *bPtr, int index) { WMMenuItem *item; + if (index >= WMGetBagItemCount(bPtr->items) || index < 0) + return NULL; + item = WMGetFromBag(bPtr->items, index); if (!item) return NULL; @@ -335,7 +338,7 @@ paintPopUpButton(PopUpButton *bPtr) if (bPtr->selectedItemIndex < 0) { /* if no item selected, show the caption */ caption = bPtr->caption; - } else { + } else { caption = WMGetPopUpButtonItem(bPtr, bPtr->selectedItemIndex); } }