commit edd7541b8ce25c1180eb5435ff3350a76d0f0b95
author Paolo Bonzini <pbonzini@redhat.com> Wed Aug 01 17:14:09 2018 +0200
committer Paolo Bonzini <pbonzini@redhat.com> Thu Aug 23 13:32:50 2018 +0200
tree 162e0c56178fb135a6ff94be70180f249fd2a458
parent 154c1d1f960c5147a3f8ef00907504112f271cd8

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>

hw/timer/sh_timer.c

diff --git a/hw/timer/sh_timer.c b/hw/timer/sh_timer.c
index 5f8736c..91b18ba 100644
--- a/hw/timer/sh_timer.c
+++ b/hw/timer/sh_timer.c
@@ -74,6 +74,7 @@
     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;