Commit 5eb7b75b authored by Michael Pyne's avatar Michael Pyne
Browse files

Correctly write TagEditor config on shutdown.

Another JuK fix from Giorgos Kylafas, this time to ensure that the tag editor
pane always correctly writes out its configuration at shutdown.

Without forcing the tag editor to be destroyed, it is possible that the KAction
the TagEditor class reads when writing its configuration would have a reset
checked state. There's probably similar bugs still extant (e.g. sometimes JuK
opens with a very wide "playlist box").

Unfortunately this comes too late to backport to 4.6.3, but I'll try for 4.6.4
once the tagging process is confirmed completed for the 4.6.3 release
(reminders always appreciated though ;)

CCMAIL:gekylafas@gmail.com

svn path=/trunk/KDE/kdemultimedia/juk/; revision=1229816
parent 700f613c
...@@ -70,6 +70,11 @@ PlaylistSplitter::~PlaylistSplitter() ...@@ -70,6 +70,11 @@ PlaylistSplitter::~PlaylistSplitter()
{ {
saveConfig(); saveConfig();
// TagEditor needs to write its configuration out while it's still valid,
// destroy it now.
delete m_editor;
// NowPlaying depends on the PlaylistCollection, so kill it now. // NowPlaying depends on the PlaylistCollection, so kill it now.
delete m_nowPlaying; delete m_nowPlaying;
m_nowPlaying = 0; m_nowPlaying = 0;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment