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

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Plasma-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to