Review Request 110057: Fix mem leaks in kunitconversion

Dario CambiƩ dariocambie at
Tue Apr 16 22:24:29 BST 2013

This is an automatically generated e-mail. To reply, visit:

(Updated April 16, 2013, 9:24 p.m.)

Review request for kdelibs.


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".


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



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/
==8754==    by 0x5A3EE11: Angle::Angle() (angle.cpp:51)

valgrind after:
no such alert


Dario CambiƩ

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list