[kde-workspace] plasma/generic/tools/plasmoidviewer: Make plasmoidviewer have non-persistent applet config files.

Aaron J. Seigo aseigo at kde.org
Wed Jan 11 12:46:39 UTC 2012


On Tuesday, January 3, 2012 04:07:48 Shaun Reich wrote:
> Git commit e5e437e9aa581056d4517c70573115582420b497 by Shaun Reich.
> Committed on 03/01/2012 at 04:04.
> Pushed by sreich into branch 'master'.
> 
> Make plasmoidviewer have non-persistent applet config files.

i'm fine with the option, but the default is wrong. one of the primary reasons 
for having it there is specifically to be able to debug configuration related 
issues.

so it should default to the pre-existing behaviour of saving and restoring 
config. deleting the config file is an easy way to reset, using the config dialog 
is often a viable and very simple optin too. so having the -pr flag[1] is 
great, the implementation looks fine, but the default is backwards.



[1] does that even work? i thought those flags had to be single character? if 
that does work: neat and kudos to KCmdLineOptions

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120111/81c9e875/attachment.sig>


More information about the Plasma-devel mailing list