----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108564/ -----------------------------------------------------------
(Updated Jan. 23, 2013, 8:51 a.m.) Review request for Telepathy. Changes ------- (Sorry for spam, did not have coffee yet) Description (updated) ------- Using Tp::ContactFactory causes crash in KTp::ContactsListModel::data(), because cast to KTp::Contact fails and then we dereference a null pointer. Question of the day: could we add something like Q_ASSERT_X(!contact.isNull(), "KTp::ContactsListModel::data()", "Cast failed. Did you forget to use Ktp::ContactFactory again?"); to data() to make it easier to spot the real problem? It took me good 15 minutes to figure out where the null pointer comes from and why the cast fails. Diffs ----- KTp/Widgets/contact-grid-dialog.cpp a23014a Diff: http://git.reviewboard.kde.org/r/108564/diff/ Testing ------- Thanks, Dan Vrátil
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
