diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2018-08-01 17:14:09 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2018-08-23 13:32:50 +0200 |
| commit | edd7541b8ce25c1180eb5435ff3350a76d0f0b95 (patch) | |
| tree | 162e0c56178fb135a6ff94be70180f249fd2a458 /hw/timer/sh_timer.c | |
| parent | 154c1d1f960c5147a3f8ef00907504112f271cd8 (diff) | |
fix "Missing break in switch" coverity reports
Many of these are marked as "intentional/fix required" because they
just need adding a fall through comment. This is exactly what this
patch does, except for target/mips/translate.c where it is easier to
duplicate the code, and hw/audio/sb16.c where I consulted the DOSBox
sources and decide to just remove the LOG_UNIMP before the fallthrough.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'hw/timer/sh_timer.c')
| -rw-r--r-- | hw/timer/sh_timer.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/timer/sh_timer.c b/hw/timer/sh_timer.c index 5f8736cf10..91b18ba312 100644 --- a/hw/timer/sh_timer.c +++ b/hw/timer/sh_timer.c @@ -74,6 +74,7 @@ static uint32_t sh_timer_read(void *opaque, hwaddr offset) case OFFSET_TCPR: if (s->feat & TIMER_FEAT_CAPT) return s->tcpr; + /* fall through */ default: hw_error("sh_timer_read: Bad offset %x\n", (int)offset); return 0; |