D20889: [kcolorschemeeditor] Remove useless printf
Kai Uwe Broulik
noreply at phabricator.kde.org
Mon Apr 29 09:41:20 BST 2019
broulik created this revision.
broulik added reviewers: Plasma, ochurlaud.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
When closing the scheme editor, the KCM reads stdout of the editor to figure out what's the saved theme so it can be selected in the list.
This printout confuses it and has it select a non-existing theme breaking the current selection.
TEST PLAN
Edited a scheme, closed editor without saving, didn't have the list reload.
Edited a scheme, saved it, closed it, list reloaded and index was still correct.
I don't think that this particular printout was used by a shell script or something?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D20889
AFFECTED FILES
kcms/colors/editor/scmeditoroptions.cpp
To: broulik, #plasma, ochurlaud
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190429/afc1e078/attachment.html>
More information about the Plasma-devel
mailing list