[Marble-devel] Review Request 121370: Fixing critical memory leaks in Pn2Runner

Illya Kovalevskyy illya.kovalevskyy at gmail.com
Sat Dec 6 19:20:57 UTC 2014


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

(Updated Dec. 6, 2014, 9:20 p.m.)


Review request for Marble, Dennis Nienhüser and Torsten Rahn.


Repository: marble


Description
-------

Valgrind detected massive memory leak (on my machine it was up to 15 MB on application launch). It was all about passing newly created object and not deleting it later. Patch closes all memory leaks from Pn2Runner.

GCI-2014 related task: http://www.google-melange.com/gci/task/view/google/gci2014/6687014871105536


Diffs (updated)
-----

  src/plugins/runner/pn2/Pn2Runner.cpp ce5dca2 

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


Testing
-------

Double valgrind run (w/o and with fixes).


Thanks,

Illya Kovalevskyy

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20141206/ca92ab83/attachment.html>


More information about the Marble-devel mailing list