Summary: | Compositing KCM does not reset OpenGLISUnsafe after a KWin crash | ||
---|---|---|---|
Product: | [Plasma] kwin | Reporter: | Luca Beltrame <lbeltrame> |
Component: | compositing | Assignee: | KWin default assignee <kwin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Flags: | mgraesslin:
ReviewRequest+
|
Priority: | NOR | ||
Version: | git master | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
URL: | https://git.reviewboard.kde.org/r/118790/ | ||
Latest Commit: | http://commits.kde.org/kwin/2088ce0e7ed1e9451cbc46532b7f5c7e0d4d037b | Version Fixed In: | |
Sentry Crash Report: |
Description
Luca Beltrame
2014-06-17 05:44:42 UTC
please specify your exact git hash as I pushed a fix for it yesterday. In data martedì 17 giugno 2014 05:47:47, hai scritto:
> please specify your exact git hash as I pushed a fix for it yesterday.
It should be 52653aa.
ok, that has the fix. Which KCM did you use? On 17 giugno 2014 08:02:00 CEST, "Martin Gräßlin " <mgraesslin@kde.org> wrote: >ok, that has the fix. Which KCM did you use? I used the desktop effects KCM (as opposed to the compositing KCM). (In reply to comment #4) > >ok, that has the fix. Which KCM did you use? > > I used the desktop effects KCM (as opposed to the compositing KCM). that explains it :-) The desktop effects KCM is not yet fixed, only the compositing KCM. I wanted to work on it today, but are not yet sure whether it should be removed altogether there. Git commit 2088ce0e7ed1e9451cbc46532b7f5c7e0d4d037b by Martin Gräßlin. Committed on 17/06/2014 at 06:30. Pushed by graesslin into branch 'master'. [kcmkwin/kwincompositing] Drop the OpenGLErrorView from Effects As we have the split between Effects and Compositor settings nowadays it's incorrect to have an Compositor related option exposed in the Effects settings view. Thus the OpenGLErrorView should be removed. As a nice side-effect that simplifies the main qml file quite a bit by turning the EffectView from Item into Rectangle. If it shows that we need the reset GL check in the effects view it would be better to use a KMessageWidget like in the compositing view. REVIEW: 118790 M +2 -1 kcmkwin/kwincompositing/qml/EffectView.qml D +0 -48 kcmkwin/kwincompositing/qml/OpenGLErrorView.qml M +4 -36 kcmkwin/kwincompositing/qml/main.qml http://commits.kde.org/kwin/2088ce0e7ed1e9451cbc46532b7f5c7e0d4d037b |