diff options
| author | Philippe Mathieu-Daudé <philmd@redhat.com> | 2019-07-19 14:40:48 +0200 |
|---|---|---|
| committer | Philippe Mathieu-Daudé <philmd@redhat.com> | 2019-07-23 11:31:07 +0200 |
| commit | 124e4cfaa42bb5a14eec33ea47d3502b5f46eb33 (patch) | |
| tree | e33d2aa7431235315d7efcf518683d6818fb79b9 /hw/block | |
| parent | 3a283507c03474d285196620fca506bd1a89b198 (diff) | |
| download | qemu-124e4cfaa42bb5a14eec33ea47d3502b5f46eb33.zip qemu-124e4cfaa42bb5a14eec33ea47d3502b5f46eb33.tar.gz qemu-124e4cfaa42bb5a14eec33ea47d3502b5f46eb33.tar.bz2 | |
hw/block/pflash_cfi02: Rewrite a fall through comment
GCC9 is confused by this comment when building with CFLAG
-Wimplicit-fallthrough=2:
hw/block/pflash_cfi02.c: In function ‘pflash_write’:
hw/block/pflash_cfi02.c:574:16: error: this statement may fall through [-Werror=implicit-fallthrough=]
574 | if (boff == 0x55 && cmd == 0x98) {
| ^
hw/block/pflash_cfi02.c:581:9: note: here
581 | default:
| ^~~~~~~
cc1: all warnings being treated as errors
Rewrite the comment using 'fall through' which is recognized by
GCC and static analyzers.
Reported-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20190719131425.10835-4-philmd@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Diffstat (limited to 'hw/block')
| -rw-r--r-- | hw/block/pflash_cfi02.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/block/pflash_cfi02.c b/hw/block/pflash_cfi02.c index f68837a..42886f6 100644 --- a/hw/block/pflash_cfi02.c +++ b/hw/block/pflash_cfi02.c @@ -577,7 +577,7 @@ static void pflash_write(void *opaque, hwaddr offset, uint64_t value, pfl->cmd = 0x98; return; } - /* No break here */ + /* fall through */ default: DPRINTF("%s: invalid write for command %02x\n", __func__, pfl->cmd); |