-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110057/
-----------------------------------------------------------

Review request for kdelibs.


Description
-------

Little patch for memory leaks in kunitconversion: functions dynamically created 
were never deleted.
Not sure if the patch is correct (C++ is not my language at all) but it seems 
to "work for me".


Diffs
-----

  kunitconversion/angle.cpp 6333925 
  kunitconversion/fuel_efficiency.cpp 1914fab 

Diff: http://git.reviewboard.kde.org/r/110057/diff/


Testing
-------

Compiles, no more mem leaks according to valgrind.

valgrind before (e.g.):
==8754== 8 bytes in 1 blocks are definitely lost in loss record 110 of 644
==8754==    at 0x4C2AF8E: operator new(unsigned long) (in 
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==8754==    by 0x5A3EE11: Angle::Angle() (angle.cpp:51)

valgrind after:
no such alert


Thanks,

Dario Cambié

Reply via email to