Review Request 127229: Workaround bug 352055

[Xuetian Weng]

> On March 1, 2016, 8:45 a.m., Marco Martin wrote:
> > Yeah, either this or changing systray for 5.6 already :/
> > and i think it's an issue in Qml, since it's very old code and this problem really started only on 5.6 branch of Qt
> 
> David Edmundson wrote:
>     This problem did not only start on the 5.6 branch of Qt. You should know that, the bug history goes back since forever.
>     
>     It's a timing/race situation. Each change makes it work for some people and makes it appear for others.

Just throw a bug report to https://bugreports.qt.io/browse/QTBUG-51583 .. maybe some one else will have some idea.


- Xuetian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127229/#review92970
-----------------------------------------------------------


On Feb. 29, 2016, 10:16 p.m., Xuetian Weng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127229/
> -----------------------------------------------------------
> 
> (Updated Feb. 29, 2016, 10:16 p.m.)
> 
> 
> Review request for Plasma and Marco Martin.
> 
> 
> Bugs: 352055
>     https://bugs.kde.org/show_bug.cgi?id=352055
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> From my debug, the real problem is that the model value in qml some times can have wrong value. When there is the "empty" space in systray, model.index obtained from delegate is -1 and no any other data is available, which should never happens.
> 
> This is obviously a bug in Qt.. but unfortunately I have no simple test case to reproduce it.
> 
> 
> Diffs
> -----
> 
>   applets/systemtray/plugin/tasklistmodel.cpp 1fa306c 
> 
> Diff: https://git.reviewboard.kde.org/r/127229/diff/
> 
> 
> Testing
> -------
> 
> Works for me at least.
> 
> 
> Thanks,
> 
> Xuetian Weng
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160301/67bf8d3d/attachment.html>