Review Request 109726: Fix crashes when using debug versions of Qt

David Edmundson david at davidedmundson.co.uk
Tue Mar 26 15:44:50 UTC 2013


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

Review request for Telepathy.


Description
-------

Fix crashes when using debug versions of Qt

Rewrite the logic of sourceDataChanged which was fundamentally broken
This was potentially emitting onDataChanged on the invalid QModelIndex,
especially now we use a flat model in the filtered view


Diffs
-----

  KTp/Models/contacts-model.cpp f472c9c538d48b8ff7b5fc63ce72e478835c71a5 
  KTp/Models/contacts-filter-model.cpp d3bfb326f54053c82f67124e7bc33278b1b77899 
  KTp/Models/contacts-filter-model.h f5b41a505058be9b55703cd7cb6787c8d399bc7e 

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


Testing
-------

Checked it no longer crashes; Online contacts count worked correctly when Martin went offline and back.


Thanks,

David Edmundson

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


More information about the KDE-Telepathy mailing list