patch for kmix kicker applet

Christian Esken c.esken at cityweb.de
Sat Jul 3 12:47:54 BST 2004


On Wednesday 30 June 2004 21:21, Koos Vriezen wrote:

>After this fix, the applet first showed 'Choose mixer', I guess that's why
>it's unnoticed.. Can someone apply this one?

Hi Koos,

thanks for the patch. I did something slightly different, but it should fix the problem, too.
It is already commited to CVS HEAD.

Chris

diff -u -r1.57 kmixapplet.cpp
--- kmixapplet.cpp      24 Jun 2004 08:53:15 -0000      1.57
+++ kmixapplet.cpp      3 Jul 2004 11:45:48 -0000
@@ -535,6 +535,10 @@

 void KMixApplet::setColors( const Colors &color )
 {
+    if ( m_mixerWidget == 0 ) {
+        // can happen for example after a paletteChange()
+        return;
+    }
     QPtrList<QWidget> &mdws = m_mixerWidget->_mdws;
     for ( QWidget* qmdw=mdws.first(); qmdw != 0; qmdw=mdws.next() ) {
        if ( qmdw->inherits("MixDeviceWidget") ) { // -<- temporary check. Later we *know* that it is correct



More information about the kde-multimedia mailing list