Review Request: Plasmate:add a destructor and some changes to the destructor of mainwindow.ccp
Giorgos Tsiapaliwkas
terietor at gmail.com
Mon Jul 11 20:12:20 CEST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101916/
-----------------------------------------------------------
Review request for Plasma.
Summary
-------
hello,
I was looking at the projectmanager.projectmanager.[h/cpp] and i noticed that there wasn't no destructor.
The patch adds a destructor at projectmanager and changed a bit the mainwindow.cpp.
I have some question in my next reply
Diffs
-----
mainwindow.cpp e08581f
projectmanager/projectmanager.h 53ef226
projectmanager/projectmanager.cpp 122e48d
Diff: http://git.reviewboard.kde.org/r/101916/diff
Testing
-------
compiles fine and no pointer error
Thanks,
Giorgos
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110711/82ed5367/attachment.htm
More information about the Plasma-devel
mailing list