diff options
| author | Thomas Jarosch <tomj@simonv.com> | 2014-12-22 21:50:47 +0100 |
|---|---|---|
| committer | Thomas Jarosch <tomj@simonv.com> | 2014-12-22 21:50:47 +0100 |
| commit | 71c0516d520ba0d05b496fb18659c0e7292d4abb (patch) | |
| tree | 9ad149d818c3141dcd482e8627715c2ef2abfb7a | |
| parent | 5d3aa1bce1ea8113f27153a0820b30d103ca35b0 (diff) | |
| download | rockbox-71c0516d520ba0d05b496fb18659c0e7292d4abb.tar.gz rockbox-71c0516d520ba0d05b496fb18659c0e7292d4abb.zip | |
zxbox: Add missing break; statement
No harm done, same values are assigned
in the "default" case.
cppcheck reported:
[rockbox/apps/plugins/zxbox/tapefile.c:674] -> [rockbox/apps/plugins/zxbox/tapefile.c:681]: (warning) Buffer 'seg_desc' is being written before its old content has been used. 'break;' missing?
Change-Id: I2bf044db9022c848c304f1ad88c012bd85ee61df
| -rw-r--r-- | apps/plugins/zxbox/tapefile.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/apps/plugins/zxbox/tapefile.c b/apps/plugins/zxbox/tapefile.c index 92f4ea69f6..b030f8dab9 100644 --- a/apps/plugins/zxbox/tapefile.c +++ b/apps/plugins/zxbox/tapefile.c | |||
| @@ -674,7 +674,8 @@ static int interpret_tzx_header(byte *hb, struct seginfo *csp) | |||
| 674 | rb->snprintf(seg_desc, DESC_LEN,"Tapefile Concatenation Point"); | 674 | rb->snprintf(seg_desc, DESC_LEN,"Tapefile Concatenation Point"); |
| 675 | csp->type = ST_MISC; | 675 | csp->type = ST_MISC; |
| 676 | csp->segtype = SEG_SKIP; | 676 | csp->segtype = SEG_SKIP; |
| 677 | 677 | break; | |
| 678 | |||
| 678 | default: | 679 | default: |
| 679 | csp->type = ST_MISC; | 680 | csp->type = ST_MISC; |
| 680 | csp->segtype = SEG_SKIP; | 681 | csp->segtype = SEG_SKIP; |