Opened 8 years ago

Last modified 8 years ago

#3954 new defect

Potential bugs from static code analysis — at Initial Version

Reported by: Henning Kiel Owned by: Lennart Ochel
Priority: normal Milestone: Future
Component: Code Generation Version: v1.10.0-dev-nightly
Keywords: Cc: Martin Sjölund, Adrian Pop

Description

I'm referencing from run "Tue Jun 7 21:05:18 2016" with 3483 warnings
(approx. commit bb33335a639ba7a14c88f997355f1d41bfe37589)

I wonder if 3rdParty code should be checked, because we cannot do anything about it, right?

[ ] 1. virtually all files
(this is not a bug, but superfluous code, slowing down C compiler)
Common problem with match expressions "Value stored to 'tmp4' is never read" (because of failure handling?) creates hundreds of warnings and makes this static analysis difficult to read

[x] 2. Absyn.mo 3584
value stored to l2 is never read -> getCrefFromExp() stop expression is lost

[ ] 3. BackendDAEOptimize.mo 1763
value stored to compInfos is never read -> info from called function is lost, function returns defaults only

[x] 4. BackendDAEOptimize.mo 879
value stored to lsteqns is never read -> BackendEquation.equationList(eqns) called twice

[ ] 5. BackendDAEOptimize.mo 4350
value stored in expLst is never read -> list obtained from right side is directly overwritten by empty list

[ ] 6. BackendDAEDump.mo 888
value stored in expLst is never read -> string with equationKindString is not used

[ ] 7. BackendEquation.mo 1066,1055,1042
use of uninitialized variable b

[ ] 8. BackendInline.mo 792
output variable outEquationArray not initialized

[ ] 9. BackendVariable.mo 2891,2882,2898
value stored in res is never read -> call to replaceVarWithWholeDimSubs() seems useless

More to come (checked Compiler/boot/build/*.c alphabetically up to and including BackendVariable.c)

Items marked with [x] are already fixed.

Change History (0)

Note: See TracTickets for help on using tickets.