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