[PATCH] Fix kmix sytray icon disappears
Christian Esken
esken at kde.org
Sun Dec 4 23:53:47 GMT 2005
Am Mittwoch, 23. November 2005 12:17 schrieb Ismail Donmez:
> Monday 21 November 2005 11:49 tarihinde, Ismail Donmez şunları yazmıştı:
> > Hi all,
> >
> > If you change any setting in kmix settings systray icon disappears.
I never had this problem. The call "mw->mixer()->readSetFromHWforceUpdate();"
in KMixWindow::applyPrefs() should make sure that all information (icon,
tooltip, context menu, ... ) will get properly refreshed.
> > Following patch fixes this. I would be glad if this makes to 3.5 release.
>
> Applied to SVN.
This patch doesn't look so good.
1) IMO it creates a memory leak, by not freeing/deleting the old
KMixDockWidget.
2) After a config dialog apply we will have two KMixDockWidget's, after the
next "apply" we have three KMixDockWidget's, and so on.
There might be several strange side effects, because of still connected
signal/slots. Comment 8 in Bug 111152
( https://bugs.kde.org/show_bug.cgi?id=111152#c8 ) might very well be
explainable by this.
3) As far as I see, it updates the pixmap, but not the tooltip (and probably
some more stuff like mute state in the context menu).
I'll try to do a checkout of KDE3.5 RC1. Any hints on reproducing this bug is
greatly appreciated.
Chris
More information about the kde-core-devel
mailing list