Review Request 113126: Fix crash caused by TextChannelWatcherProxyModel

David Edmundson david at davidedmundson.co.uk
Sun Oct 6 15:22:28 UTC 2013


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

(Updated Oct. 6, 2013, 3:22 p.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy.


Repository: ktp-common-internals


Description
-------

Fix crash caused by TextChannelWatcherProxyModel

void KTp::TextChannelWatcherProxyModel::onChannelMessagesChanged()
void KTp::TextChannelWatcherProxyModel::onChannelInvalidated()
emit dataChanged() using indexes from the wrong model.

Because it's an QIdentityProxyModel it never causes any harm,
but it's still "wrong" so Qt asserts break in the grouping model.

BUG: 325679


Diffs
-----

  KTp/Models/text-channel-watcher-proxy-model.cpp ea863fd 

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


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20131006/bc89675f/attachment.html>


More information about the KDE-Telepathy mailing list