- less qimage<>qpixmap conversions

1 files changed, 6 insertions, 5 deletions

diff --git a/noncore/multimedia/opieplayer2/mediawidget.cpp b/noncore/multimedia/opieplayer2/mediawidget.cpp
index 865ff91..70ef8d9 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.cpp
+++ b/noncore/multimedia/opieplayer2/mediawidget.cpp
@@ -67,30 +67,31 @@ MediaWidget::Button MediaWidget::setupButton( const SkinButtonInfo &buttonInfo,
 
     button.mask = setupButtonMask( button.command, fileName );
 
     return button;
 }
 
 QBitmap MediaWidget::setupButtonMask( const Command &command, const QString &fileName )
 {
-    QBitmap mask( Resource::findPixmap( fileName ) );
-    if ( mask.isNull() )
-        return mask;
+    QImage imgMask( Resource::findPixmap( fileName ) );
+    if ( imgMask.isNull() )
+        return QBitmap();
 
-    QImage imgMask = mask.convertToImage();
     uchar **dest = buttonMask.jumpTable();
     for ( int y = 0; y < buttonMask.height(); y++ ) {
         uchar *line = dest[y];
         for ( int x = 0; x < buttonMask.width(); x++ )
             if ( !qRed( imgMask.pixel( x, y ) ) )
                 line[x] = command + 1;
     }
 
-    return mask;
+    // ### grmbl qt2. use constructor when switching to qt3.
+    QBitmap bm; bm = imgMask;
+    return bm;
 }
 
 void MediaWidget::loadDefaultSkin( const SkinButtonInfo *skinInfo, uint buttonCount, const QString &fileNameInfix )
 {
     Config cfg( "OpiePlayer" );
     cfg.setGroup( "Options" );
     QString skin = cfg.readEntry( "Skin","default" );