[Scummvm-devel] coverity

[Willem Jan Palenstijn]

Hi all,

I just want to remind everyone that "fixing" issues just because they offend
Coverity's sense of style is not always a good idea.

If things are false positives, please mark them as false positives or to be
ignored in some other way instead.


In particular, blindly initializing variables in constructors while they
_should_ already be initialized in some init() method just hides potential bugs
in init() or missed calls to init(). On top of that, it makes tools like
valgrind or better static analysis useless for finding and catching any such bugs.


-Willem Jan