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