Review Request: Delete ChatWidgets when removed from ChatWindow's tab bar

Lasse Liehu lliehu at kolumbus.fi
Thu Mar 3 00:38:29 CET 2011


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

Review request for Telepathy.


Summary
-------

When a tab was removed from chat window, the widget itself wasn't deleted, even though it seems it should have been.

As I don't know the code base that well, I feel insecure about if there are any pointers to ChatWidgets elsewhere than in the tab bar. Probably not, but still.


Diffs
-----

  app/chatwindow.h 663a54a 
  app/chatwindow.cpp fed4381 

Diff: http://git.reviewboard.kde.org/r/100793/diff


Testing
-------

Compiles and doesn't crash when a tab is removed by clicking on its close button.


Thanks,

Lasse

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110302/4edd0fd1/attachment.html 


More information about the KDE-Telepathy mailing list