Bug 177178 - Kwin KCM module creates invalid kwinrc file
Summary: Kwin KCM module creates invalid kwinrc file
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-07 22:31 UTC by Adam Pigg
Modified: 2008-12-08 06:08 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
My kwinrc on login (1.73 KB, text/plain)
2008-12-07 22:33 UTC, Adam Pigg
Details
kwinrc when apply the shadow effect from the general tab (1.73 KB, text/plain)
2008-12-07 22:34 UTC, Adam Pigg
Details
kwinrc when apply wobbly window on the advanced tab (2.89 KB, text/plain)
2008-12-07 22:34 UTC, Adam Pigg
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Pigg 2008-12-07 22:31:41 UTC
Version:           4.1.82 (using Devel)
OS:                Linux
Installed from:    Compiled sources

Between 4.8.81 and 4.1.82, something caused the kwinrc file to become corrupted when settings are applied in the kwin kcm module.

From the user perspective it appears that settings arnt being saved, for example, selecting the wobbly window plugin, clicking ok, then going back in shows the wobblywindow not selected.

Looking at kwinrc though, almost every setting is duplicated, and here you can see that wobbly window is both true and false.
Comment 1 Adam Pigg 2008-12-07 22:33:24 UTC
Created attachment 29128 [details]
My kwinrc on login
Comment 2 Adam Pigg 2008-12-07 22:34:04 UTC
Created attachment 29129 [details]
kwinrc when apply the shadow effect from the general tab
Comment 3 Adam Pigg 2008-12-07 22:34:49 UTC
Created attachment 29130 [details]
kwinrc when apply wobbly window on the advanced tab

This is where the duplicates are visible
Comment 4 lucas 2008-12-08 06:08:34 UTC
SVN commit 894182 by lmurray:

Reverted all code relating to automatic X crash detection with agreement
from David. Due to an elusive bug somewhere in kdelibs we have not been
able to find out how to fix the issues that some people were having.
There seems to be a race condition as I cannot reproduce at all, David
can reproduce sporadically and some reporters can always reproduce.

This commit attempts to also keep the code style changes that were made
at the same time as the original commit as well as some minor bug fixes
that were made between then and now. Hopefully I didn't miss anything.

We will be revisit this in KDE 4.3.

BUG: 177178

 M  +148 -241  kcmkwin/kwincompositing/main.cpp  
 M  +14 -19    kcmkwin/kwincompositing/main.h  
 M  +35 -72    options.cpp  
 M  +1 -7      options.h  
 M  +4 -1      org.kde.KWin.xml  
 M  +19 -4     workspace.cpp  
 M  +3 -2      workspace.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=894182