diff options
| author | Simon Glass <sjg@chromium.org> | 2023-01-06 08:52:25 -0600 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2023-01-16 14:14:11 -0500 |
| commit | 86cc3c5215fc6e3c2cb77ee162c22ad91dbfaff5 (patch) | |
| tree | ca29c6c8479815a5140e91dc273ed3b698bf4909 /common | |
| parent | d0ca98dbd99c2534c9e96e4c226e52ab80f2248f (diff) | |
| download | u-boot-86cc3c5215fc6e3c2cb77ee162c22ad91dbfaff5.zip u-boot-86cc3c5215fc6e3c2cb77ee162c22ad91dbfaff5.tar.gz u-boot-86cc3c5215fc6e3c2cb77ee162c22ad91dbfaff5.tar.bz2 | |
menu: Use a switch statement
Convert the long line of if() statements to a switch() since this makes
better use of the C language.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/menu.c | 31 |
1 files changed, 20 insertions, 11 deletions
diff --git a/common/menu.c b/common/menu.c index 6842f54..7db9894 100644 --- a/common/menu.c +++ b/common/menu.c @@ -543,22 +543,31 @@ enum bootmenu_key bootmenu_loop(struct bootmenu_data *menu, int *esc) break; } - /* enter key was pressed */ - if (c == '\r') + switch (c) { + case '\r': + /* enter key was pressed */ key = BKEY_SELECT; - - /* ^C was pressed */ - if (c == 0x3) + break; + case CTL_CH('c'): + /* ^C was pressed */ key = BKEY_QUIT; - - if (c == '+') + break; + case CTL_CH('p'): + key = BKEY_UP; + break; + case CTL_CH('n'): + key = BKEY_DOWN; + break; + case '+': key = BKEY_PLUS; - - if (c == '-') + break; + case '-': key = BKEY_MINUS; - - if (c == ' ') + break; + case ' ': key = BKEY_SPACE; + break; + } return key; } |