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

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to