From 2cc84d307befbd45f5e58338c27969cfbc415944 Mon Sep 17 00:00:00 2001 From: Javier Serrano Polo Date: Mon, 7 Aug 2006 20:29:15 +0000 Subject: [PATCH] - added manualChange signal - emit valueChanged whenever value is changed - removed redundant journalling operations git-svn-id: https://lmms.svn.sf.net/svnroot/lmms/trunk/lmms@286 0778d3d1-df1d-0410-868b-ea421aaaa00d --- src/widgets/lcd_spinbox.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/widgets/lcd_spinbox.cpp b/src/widgets/lcd_spinbox.cpp index a19e9d14f..017373060 100644 --- a/src/widgets/lcd_spinbox.cpp +++ b/src/widgets/lcd_spinbox.cpp @@ -113,6 +113,7 @@ void lcdSpinBox::setStep( const int _step ) void lcdSpinBox::setValue( const int _value ) { + const int prev_value = value(); autoObj::setValue( _value ); QString s = m_textForValue[value()]; if( s == "" ) @@ -124,6 +125,11 @@ void lcdSpinBox::setValue( const int _value ) } } m_number->display( s ); + + if( prev_value != value() ) + { + emit valueChanged( value() ); + } } @@ -231,10 +237,8 @@ void lcdSpinBox::mouseMoveEvent( QMouseEvent * _me ) int dy = _me->globalY() - m_origMousePos.y(); if( dy > 1 || dy < -1 ) { - setJournalling( FALSE );// why is this neccessary?! setInitValue( value() - dy / 2 * step() ); - emit valueChanged( value() ); - setJournalling( TRUE ); + emit manualChange(); QCursor::setPos( m_origMousePos ); } } @@ -258,7 +262,7 @@ void lcdSpinBox::wheelEvent( QWheelEvent * _we ) { _we->accept(); setInitValue( value() + ( ( _we->delta() > 0 ) ? 1 : -1 ) * step() ); - emit valueChanged( value() ); + emit manualChange(); }