Bug 344684 - QML binding loop in: org.kde.breeze.desktop/contents/osd/OsdItem.qml:51:5: QML ProgressBar: Binding loop detected for property "value"
Summary: QML binding loop in: org.kde.breeze.desktop/contents/osd/OsdItem.qml:51:5: QM...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Theme - Breeze (show other bugs)
Version: master
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: 1.0
Assignee: Andrew Lake
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-28 22:55 UTC by Elias Probst
Modified: 2015-03-22 18:50 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Elias Probst 2015-02-28 22:55:16 UTC
On plasmashell startup, the following QML binding loop is logged:

default unknown: file:///usr/share/plasma/look-and-feel/org.kde.breeze.desktop/contents/osd/OsdItem.qml:51:5: QML ProgressBar: Binding loop detected for property "value"

Running latest KF5 (git) + Plasma 5 (git) + Qt 5.4.1.
Comment 1 Marco Martin 2015-03-20 10:37:10 UTC
hmm, this binding loop doesn't seem to make much sense, may be a false positive
Comment 2 David Edmundson 2015-03-22 18:47:03 UTC
Confirming binding loop exists.
Comment 3 David Edmundson 2015-03-22 18:50:25 UTC
Git commit 770b5dd69dd6313df31d4f52d00723da01642abc by David Edmundson.
Committed on 22/03/2015 at 19:37.
Pushed by davidedmundson into branch 'master'.

Sanitise input to osd progress bar

Previous code checked the progress bar was visible and only then set the
osdValue (which is a var). This relies on inner workings of the OSD
service to set values in the right order.

This fixes a binding loop on startup and will be slightly faster as we
won't update value twice when showing OSD.

M  +1    -1    lookandfeel/contents/osd/OsdItem.qml

http://commits.kde.org/plasma-workspace/770b5dd69dd6313df31d4f52d00723da01642abc