D25609: Port KSMServer's KCM to KConfigXT

[Kevin Ottens]

ervin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in kcmsmserver.cpp:203
> This patch renames this entry "default" to "emptySession"
> 
> From what I can tell it won't matter for ksmserver itself as that only compares against the other two strings.
> 
> But if  a user had this value set currently then opens the KCM afterwards, which value gets loaded?

Yes, several reasons for that change: default doesn't play well as an enum value, also it wasn't the default at all... since restoring the previous session is.
In our case, if there's "default" in the file, it will in practice fallback to restore previous session which I think is fine as a behavior.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D25609

To: ervin, #plasma, mart, davidedmundson, crossi, bport, mlaurent
Cc: mlaurent, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191129/69fdb6c5/attachment.html>