Ticket #3467 (closed defect: fixed)
[patch] cleanup -Wunreachable-code warnings
Reported by: | and | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 4.8.15 |
Component: | mc-core | Version: | master |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Branch state: | merged | Votes for changeset: |
Description
fix -Wunreachable-code warnings and add to mc cflags
input_complete.c:1170:9: warning: 'break' will never be executed [-Wunreachable-code-break]
ascii.c:291:12: error: 'return' will never be executed [-Wunreachable-code-return]
cons.saver.c:211:9: warning: 'break' will never be executed [-Wunreachable-code-break]
Signed-off-by: Andreas Mohr <and@…>
Attachments
Change History
comment:2 follow-up: ↓ 3 Changed 9 years ago by zaytsev
and, are you interested in triaging our Coverity report? Just thought you might find it funny since you're systematically checking the code for non-default warnings. If yes, then please register and apply for access to our project:
https://scan.coverity.com/projects/1912
I've uploaded a build of master branch a couple of days ago, so the data is still fresh.
andrew_b, are you applying this in master or a cleanup branch? Do you think it would make sense for me to submit the cleanup branch to Coverity instead, if and is interested to avoid triaging what has already been fixed?
comment:3 in reply to: ↑ 2 Changed 9 years ago by andrew_b
Replying to zaytsev:
andrew_b, are you applying this in master or a cleanup branch?
In 3420_cleanup branch.
Do you think it would make sense for me to submit the cleanup branch to Coverity instead,
Yes, I do.
comment:4 Changed 9 years ago by andrew_b
- Status changed from new to closed
- Resolution set to fixed
- Branch state changed from no branch to merged
- Milestone changed from Future Releases to 4.8.15
Thanks!
Applied as [f023a83502ceef05ee937725b0ac14bbdf082775].
comment:6 Changed 4 years ago by and
Hi zaytev,
can you approve my request at
https://scan.coverity.com/projects/1912
comment:7 Changed 4 years ago by zaytsev
Hi Andreas,
I hope I did it right - if I have to upload something for you, let me know.
Yury
comment:8 Changed 4 years ago by and
Thanks, now I see more.
Last run from 2014 is too old.
Can you/we(?) bump against current 4103_cleanup branch?
Looks like only admin(?) can "Submit build" at coverity Project Settings?
Or if I know the specific project coverity-token (from .travis.yml) I can trigger by myself,
by a simple curl request.
comment:9 Changed 4 years ago by zaytsev
Hi Andreas,
I've just sent you an email.
All the best,
Yury
comment:10 Changed 4 years ago by and
Thanks,
uploaded 4.8.25-99-gf9cce0d3a,
let see when Coverity dashboard is updated
comment:11 Changed 4 years ago by zaytsev
Last Build Status: Failed. Your build has failed due to the following reason. Please fix the error and upload the build again.
comment:12 Changed 4 years ago by and
mhh, are there error details?
comment:13 Changed 4 years ago by and
fixed, tar content path was not relative.