[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-multimedia
Subject:    Re: patch for kmix kicker applet
From:       Christian Esken <c.esken () cityweb ! de>
Date:       2004-07-03 11:47:54
Message-ID: 200407031347.54541.c.esken () cityweb ! de
[Download RAW message or body]

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

_______________________________________________
kde-multimedia mailing list
kde-multimedia@kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia


[prev in list] [next in list] [prev in thread] [next in thread]