[Digikam-devel] [digikam] [Bug 326525] Severe memory leak during maintainance scan for rebuild thumbnails

[Gilles Caulier]

https://bugs.kde.org/show_bug.cgi?id=326525

--- Comment #65 from Gilles Caulier <caulier.gilles at gmail.com> ---
Raphael,

There are few warnings reported by cppcheck. Nothing urgent, just informative
and good to know :

[gilles at localhost libpgf]$ cppcheck --enable=all .
...
[Decoder.h:46]: (style) The class 'CDecoder' does not have a constructor.
[Decoder.cpp:499]: (style) Throwing a copy of the caught exception instead of
rethrowing the original exception.
[Decoder.cpp:536]: (warning) Redundant assignment of 'wordLen' to itself.
[Decoder.cpp:595]: (warning) Redundant assignment of 'wordLen' to itself.
[BitStream.h:129]: (style) The scope of the variable 'hiCount' can be reduced.
[Decoder.h:46]: (style) 'class CDecoder' does not have a copy constructor which
is recommended since the class contains a pointer to allocated memory.
...
[Encoder.h:46]: (style) The class 'CEncoder' does not have a constructor.
[Encoder.cpp:405]: (style) Variable 'h' is assigned a value that is never used.
[Encoder.cpp:744]: (style) The scope of the variable 'cnt' can be reduced.
[Encoder.cpp:570]: (style) Unsigned variable 'wordPos' can't be negative so it
is unnecessary to test it.
[Encoder.cpp:583]: (style) Unsigned variable 'wordPos' can't be negative so it
is unnecessary to test it.
[Encoder.cpp:593]: (style) Unsigned variable 'wordPos' can't be negative so it
is unnecessary to test it.
[Encoder.cpp:605]: (style) Unsigned variable 'wordPos' can't be negative so it
is unnecessary to test it.
[Encoder.cpp:614]: (style) Unsigned variable 'm_codePos' can't be negative so
it is unnecessary to test it.
[Encoder.cpp:769]: (style) Unsigned variable 'codePos' can't be negative so it
is unnecessary to test it.
[Encoder.h:46]: (style) 'class CEncoder' does not have a copy constructor which
is recommended since the class contains a pointer to allocated memory.
...
[PGFimage.cpp:1294]: (error) syntax error
[PGFimage.cpp:637]: (style) Unsigned variable 'len' can't be negative so it is
unnecessary to test it.
[PGFimage.cpp:680]: (style) Unsigned variable 'len' can't be negative so it is
unnecessary to test it.
...
[gilles at localhost libpgf]$ 

Gilles Caulier

-- 
You are receiving this mail because:
You are the assignee for the bug.