seqevent.cpp: bad switch statements ?
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Seq24 |
New
|
Undecided
|
Unassigned | ||
Bug Description
I just ran the static analyser "cppcheck" over the source code of
seq24-0.9.3
it said
[seqevent.cpp:562] -> [seqevent.cpp:564]: (warning) Variable 'result' is reassigned a value before the old one has been used. 'break;' missing?
[seqevent.cpp:564] -> [seqevent.cpp:566]: (warning) Variable 'result' is reassigned a value before the old one has been used. 'break;' missing?
[seqevent.cpp:606] -> [seqevent.cpp:608]: (warning) Variable 'result' is reassigned a value before the old one has been used. 'break;' missing?
[seqevent.cpp:608] -> [seqevent.cpp:610]: (warning) Variable 'result' is reassigned a value before the old one has been used. 'break;' missing?
[seqevent.cpp:625] -> [seqevent.cpp:627]: (warning) Variable 'result' is reassigned a value before the old one has been used. 'break;' missing?
[seqevent.cpp:627] -> [seqevent.cpp:629]: (warning) Variable 'result' is reassigned a value before the old one has been used. 'break;' missing?
Suggest add some break statements.
