diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2018-08-05 00:18:25 -0400 |
---|---|---|
committer | Daniel Thompson <daniel.thompson@linaro.org> | 2018-11-13 15:38:50 -0500 |
commit | 01cb37351bafc1b44b962842926210115e231f0a (patch) | |
tree | 2890316d57566cbe95fdc5c1616ecbbad6492f42 | |
parent | 9eb62f0e1bc70ebc9b15837a0c4e8f12a7b910cb (diff) |
kdb: kdb_keyboard: mark expected switch fall-throughs
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Notice that in this particular case, I replaced the code comments with
a proper "fall through" annotation, which is what GCC is expecting
to find.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
-rw-r--r-- | kernel/debug/kdb/kdb_keyboard.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/debug/kdb/kdb_keyboard.c b/kernel/debug/kdb/kdb_keyboard.c index 118527aa60ea..750497b0003a 100644 --- a/kernel/debug/kdb/kdb_keyboard.c +++ b/kernel/debug/kdb/kdb_keyboard.c | |||
@@ -173,11 +173,11 @@ int kdb_get_kbd_char(void) | |||
173 | case KT_LATIN: | 173 | case KT_LATIN: |
174 | if (isprint(keychar)) | 174 | if (isprint(keychar)) |
175 | break; /* printable characters */ | 175 | break; /* printable characters */ |
176 | /* drop through */ | 176 | /* fall through */ |
177 | case KT_SPEC: | 177 | case KT_SPEC: |
178 | if (keychar == K_ENTER) | 178 | if (keychar == K_ENTER) |
179 | break; | 179 | break; |
180 | /* drop through */ | 180 | /* fall through */ |
181 | default: | 181 | default: |
182 | return -1; /* ignore unprintables */ | 182 | return -1; /* ignore unprintables */ |
183 | } | 183 | } |