https://bugs.kde.org/show_bug.cgi?id=419625

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/ksc
                   |                            |reen/cfd95e6bff4bfeadf95a7a
                   |                            |475693b30ddfe30431
         Resolution|---                         |FIXED

--- Comment #9 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit cfd95e6bff4bfeadf95a7a475693b30ddfe30431 by David Edmundson.
Committed on 02/05/2020 at 21:52.
Pushed by davidedmundson into branch 'master'.

fix(kcm): avoid recursive OpacityMask which leads to teardown crash

Summary:
In the current code OpacityMask's source is the parent which contains
itself, which doesn't make sense. Docs explicitly say this. [1]

This leads to a crash in teardown.

A quick reshuffle fixes it.

[1] https://doc.qt.io/qt-5/qml-qtgraphicaleffects-opacitymask.html#source-prop

Test Plan:
Found reproducible steps (see bug)
No longer crashes

Reviewers: romangg

Reviewed By: romangg

Subscribers: romangg, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D29340

M  +15   -11   kcm/package/contents/ui/Output.qml

https://commits.kde.org/kscreen/cfd95e6bff4bfeadf95a7a475693b30ddfe30431

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to