From 918fed9966764d49c5ffe267190be713012c52c1 Mon Sep 17 00:00:00 2001 From: Laurent MONIN Date: Mon, 26 Dec 2005 10:53:39 +0100 Subject: [PATCH] kbd_listbox(): converts if {} stuff to a switch{}. --- src/bfu/listbox.c | 30 +++++++++++++++++------------- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/src/bfu/listbox.c b/src/bfu/listbox.c index 1ce22c7e2..fc7aac16c 100644 --- a/src/bfu/listbox.c +++ b/src/bfu/listbox.c @@ -622,22 +622,21 @@ kbd_listbox(struct dialog_data *dlg_data, struct widget_data *widget_data) case EVENT_KBD: action_id = kbd_action(KEYMAP_MENU, ev, NULL); - /* Moving the box */ - if (action_id == ACT_MENU_DOWN) { + switch (action_id) { + case ACT_MENU_DOWN: listbox_sel_move(dlg_item, 1); display_widget(dlg_data, dlg_item); return EVENT_PROCESSED; - } - if (action_id == ACT_MENU_UP) { + case ACT_MENU_UP: listbox_sel_move(dlg_item, -1); display_widget(dlg_data, dlg_item); return EVENT_PROCESSED; - } - if (action_id == ACT_MENU_PAGE_DOWN) { + case ACT_MENU_PAGE_DOWN: + { struct listbox_data *box; box = get_listbox_widget_data(dlg_item); @@ -651,7 +650,8 @@ kbd_listbox(struct dialog_data *dlg_data, struct widget_data *widget_data) return EVENT_PROCESSED; } - if (action_id == ACT_MENU_PAGE_UP) { + case ACT_MENU_PAGE_UP: + { struct listbox_data *box; box = get_listbox_widget_data(dlg_item); @@ -665,21 +665,20 @@ kbd_listbox(struct dialog_data *dlg_data, struct widget_data *widget_data) return EVENT_PROCESSED; } - if (action_id == ACT_MENU_HOME) { + case ACT_MENU_HOME: listbox_sel_move(dlg_item, -INT_MAX); display_widget(dlg_data, dlg_item); return EVENT_PROCESSED; - } - if (action_id == ACT_MENU_END) { + case ACT_MENU_END: listbox_sel_move(dlg_item, INT_MAX); display_widget(dlg_data, dlg_item); return EVENT_PROCESSED; - } - if (action_id == ACT_MENU_MARK_ITEM) { + case ACT_MENU_MARK_ITEM: + { struct listbox_data *box; box = get_listbox_widget_data(dlg_item); @@ -692,7 +691,8 @@ kbd_listbox(struct dialog_data *dlg_data, struct widget_data *widget_data) return EVENT_PROCESSED; } - if (action_id == ACT_MENU_DELETE) { + case ACT_MENU_DELETE: + { struct listbox_data *box; box = get_listbox_widget_data(dlg_item); @@ -705,6 +705,10 @@ kbd_listbox(struct dialog_data *dlg_data, struct widget_data *widget_data) return EVENT_PROCESSED; } + default: + return EVENT_NOT_PROCESSED; + } + /* Selecting a button; most probably ;). */ break;