- moved the duplicated code for checking whether the mouse is over a

  button into a shared isOverButton method

1 files changed, 1 insertions, 6 deletions

diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index 4301a67..d083273 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -381,12 +381,7 @@ void AudioWidget::mouseMoveEvent( QMouseEvent *event ) {
     for ( unsigned int i = 0; i < buttons.size(); i++ ) {
         if ( event->state() == QMouseEvent::LeftButton ) {
             // The test to see if the mouse click is inside the button or not
-            int x = event->pos().x() - upperLeftOfButtonMask.x();
-            int y = event->pos().y() - upperLeftOfButtonMask.y();
-
-            bool isOnButton = ( x > 0 && y > 0 && x < buttonMask.width()
-                                && y < buttonMask.height()
-                                && buttonMask.pixelIndex( x, y ) == i + 1 );
+            bool isOnButton = isOverButton( event->pos() - upperLeftOfButtonMask, i );
 
             if ( isOnButton && !buttons[i].isHeld ) {
                 buttons[i].isHeld = TRUE;