Review Request 115983: Reduce memory leaks

Kevin Krammer krammer at kde.org
Sun Feb 23 22:14:18 UTC 2014


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

(Updated Feb. 23, 2014, 10:14 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Chusslove Illich.


Repository: ki18n


Description
-------

Create the script engine as a QObject child of the interface and
delete all interfaces in KTranscriptImp's destructor.

valgrind --tool=memcheck ./ktranscripttest

before:


==10664== HEAP SUMMARY:
==10664==     in use at exit: 445,913 bytes in 2,753 blocks
==10664==   total heap usage: 27,995 allocs, 25,242 frees, 6,059,328 bytes allocated
==10664== 
==10664== LEAK SUMMARY:
==10664==    definitely lost: 0 bytes in 0 blocks
==10664==    indirectly lost: 0 bytes in 0 blocks
==10664==      possibly lost: 1,488 bytes in 3 blocks
==10664==    still reachable: 444,425 bytes in 2,750 blocks
==10664==         suppressed: 0 bytes in 0 blocks


after: 

==11788== HEAP SUMMARY:
==11788==     in use at exit: 13,778 bytes in 66 blocks
==11788==   total heap usage: 28,003 allocs, 27,937 frees, 6,064,040 bytes allocated
==11788== 
==11788== LEAK SUMMARY:
==11788==    definitely lost: 0 bytes in 0 blocks
==11788==    indirectly lost: 0 bytes in 0 blocks
==11788==      possibly lost: 1,488 bytes in 3 blocks
==11788==    still reachable: 12,290 bytes in 63 blocks
==11788==         suppressed: 0 bytes in 0 blocks


Diffs
-----

  src/ktranscript.cpp 1ce0d1a 

Diff: https://git.reviewboard.kde.org/r/115983/diff/


Testing
-------

All tests still run successfully


Thanks,

Kevin Krammer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140223/bffaa56b/attachment.html>


More information about the Kde-frameworks-devel mailing list