Review Request 108830: Make Qt::DisplayRole compare the same as default

[Commit Hook]

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


This review has been submitted with commit 27891ac314b1fd71517070241dbc97019433a75c by Martin Klapetek to branch master.

- Commit Hook


On Feb. 7, 2013, 1:03 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108830/
> -----------------------------------------------------------
> 
> (Updated Feb. 7, 2013, 1:03 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Comparing the Qt::DisplayRole is basically the same as comparing default, except we're querying the model once more even if we already have the needed strings, so let's reuse that. Also fixes a huge and super annoying problem when the Qt::DisplayRole was comparing the same strings.
> 
> 
> Diffs
> -----
> 
>   KTp/Models/contacts-filter-model.cpp f0372b4 
> 
> Diff: http://git.reviewboard.kde.org/r/108830/diff/
> 
> 
> Testing
> -------
> 
> Yes.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

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