diff options
| author | Stefan Weil <sw@weilnetz.de> | 2012-01-09 18:29:51 +0100 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> | 2012-01-13 10:36:59 +0000 |
| commit | 0b0404bf84432f0ee74840d674b93bacee7eee68 (patch) | |
| tree | f8ba2feae78afdda61a37ecdd9de413e962318e1 /qemu-option.c | |
| parent | 3c8359d11aee9d3fdcab0f184605f603f91f34f7 (diff) | |
Add 'fall through' comments to case statements without break
These comments are used by static code analysis tools and in code reviews
to avoid false warnings because of missing break statements.
The case statements handled here were reported by coverity.
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Diffstat (limited to 'qemu-option.c')
| -rw-r--r-- | qemu-option.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/qemu-option.c b/qemu-option.c index 6b23c3123..a303f87e1 100644 --- a/qemu-option.c +++ b/qemu-option.c @@ -214,13 +214,17 @@ static int parse_option_size(const char *name, const char *value, uint64_t *ret) switch (*postfix) { case 'T': sizef *= 1024; + /* fall through */ case 'G': sizef *= 1024; + /* fall through */ case 'M': sizef *= 1024; + /* fall through */ case 'K': case 'k': sizef *= 1024; + /* fall through */ case 'b': case '\0': *ret = (uint64_t) sizef; |