Closed GoogleCodeExporter closed 9 years ago
I do not see why ?
Original comment by mathieu.malaterre
on 7 Mar 2014 at 10:09
It forces the code to check all three case statements without a break.
Also, looks like perhaps it was forgotten.
Best practice is to add a break.
Original comment by boxe...@gmail.com
on 7 Mar 2014 at 1:14
Please either:
- provide a test case to understand what is wrong with this *valid* code
- or point to the section in part1 you think is incorrect.
From here, it looks as if you are reporting some kind of warning from your
compiler.
Marking as wontfix for now.
Original comment by mathieu.malaterre
on 7 Mar 2014 at 2:29
Hi Mathieu,
IMHO, the less warnings the better. After more than, say 20 compiler warnings,
one tends to not look too carefully at them, and one might miss an actual bug.
Aaron
Original comment by boxe...@gmail.com
on 7 Mar 2014 at 3:52
All 20 OS (Win/Lin/Mac) & Compilers (clang, gcc, VS) configurations are here:
http://my.cdash.org/index.php?project=OPENJPEG
Where do you see this warning ?
Original comment by mathieu.malaterre
on 7 Mar 2014 at 4:02
I am using Eclipse CDT Codan:
https://wiki.eclipse.org/CDT/designs/StaticAnalysis
Original comment by boxe...@gmail.com
on 7 Mar 2014 at 4:36
Codan gives me about 22 warnings
Original comment by boxe...@gmail.com
on 7 Mar 2014 at 4:42
Original issue reported on code.google.com by
boxe...@gmail.com
on 5 Mar 2014 at 3:25