diff options
| author | Philippe Mathieu-Daudé | 2019-07-19 15:14:19 +0200 |
|---|---|---|
| committer | Laurent Vivier | 2019-08-21 10:54:31 +0200 |
| commit | 6f0dd6c5d25a0355b1dfc36dd8ed40dc053b3d54 (patch) | |
| tree | 4a9e81711907354410926ba9112cf251f6234743 /qobject | |
| parent | hw/net/e1000: Fix erroneous comment (diff) | |
| download | qemu-6f0dd6c5d25a0355b1dfc36dd8ed40dc053b3d54.tar.gz qemu-6f0dd6c5d25a0355b1dfc36dd8ed40dc053b3d54.tar.xz qemu-6f0dd6c5d25a0355b1dfc36dd8ed40dc053b3d54.zip | |
json: Move switch 'fall through' comment to correct place
Reported by GCC9 when building with CFLAG -Wimplicit-fallthrough=2:
qobject/json-parser.c: In function ‘parse_literal’:
qobject/json-parser.c:492:24: error: this statement may fall through [-Werror=implicit-fallthrough=]
492 | case JSON_INTEGER: {
| ^
qobject/json-parser.c:524:5: note: here
524 | case JSON_FLOAT:
| ^~~~
Correctly place the 'fall through' comment.
Reported-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190719131425.10835-2-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'qobject')
| -rw-r--r-- | qobject/json-parser.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qobject/json-parser.c b/qobject/json-parser.c index 7d23e12e33..d083810d37 100644 --- a/qobject/json-parser.c +++ b/qobject/json-parser.c @@ -519,8 +519,8 @@ static QObject *parse_literal(JSONParserContext *ctxt) } assert(ret == -ERANGE); } - /* fall through to JSON_FLOAT */ } + /* fall through to JSON_FLOAT */ case JSON_FLOAT: /* FIXME dependent on locale; a pervasive issue in QEMU */ /* FIXME our lexer matches RFC 8259 in forbidding Inf or NaN, |