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