----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113126/#review42830 -----------------------------------------------------------
This review has been submitted with commit d5b5ed3967050e5715a341d5d2ce36bf275a2e45 by David Edmundson to branch kde-telepathy-0.7. - Commit Hook On Oct. 6, 2013, 3:22 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > 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.) > > > 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
