make timerStart called twice just reset, instead of stop

2 files changed, 28 insertions, 21 deletions

diff --git a/noncore/tools/clock/clock.cpp b/noncore/tools/clock/clock.cpp
index 9898332..0eb2b83 100644
--- a/noncore/tools/clock/clock.cpp
+++ b/noncore/tools/clock/clock.cpp
@@ -220,115 +220,126 @@ void Clock::updateClock()
 					if ( hour == 0 )
 						hour = 12;
 					if ( hour > 12 )
 						hour -= 12;
 					s.sprintf( "%2d%c%02d", hour, ':', tm.minute() );
 					ampmLabel->setText( ( tm.hour() >= 12 ) ? "PM" : "AM" );
 					ampmLabel->show();
 				}
 			else
 				{
 					s.sprintf( "%2d%c%02d", tm.hour(), ':', tm.minute() );
 					ampmLabel->hide();
 				}
 			lcd->display( s );
 			lcd->repaint( FALSE );
 			aclock->display( QTime::currentTime() );
 			date->setText( TimeString::longDateString( QDate::currentDate() ) );
 		}
   else
 		{
 			QTime swatch_time;
 			QString lcdtext;
 			int totalms = swatch_totalms;
 			if ( swatch_running )
 				totalms += swatch_start.elapsed();
 			swatch_time = QTime( 0, 0, 0 ).addMSecs( totalms );
 			QString d = swatch_running ? QString( "    " )
 									: QString::number( totalms % 1000 + 1000 );
 			lcdtext = swatch_time.toString() + "." + d.right( 3 ).left( sw_prec );
 			lcd->display( lcdtext );
 			lcd->repaint( FALSE );
 			aclock->display( swatch_time );
 			date->setText( TimeString::longDateString( QDate::currentDate() ) );
 		}
 }
 
 void Clock::changeClock( bool a )
 {
   ampm = a;
   updateClock();
 }
 
 void Clock::clearClock( void )
 {
   lcd->display( QTime( 0, 0, 0 ).toString() );
   aclock->display( QTime( 0, 0, 0 ) );
 }
 
+void Clock::startSWatch()
+{
+		swatch_start.start();
+		set->setText( tr( "Stop" ) );
+		t->start( 1000 );
+		swatch_running = TRUE;
+			// disable screensaver while stop watch is running
+		toggleScreenSaver( FALSE );
+}
+
+void Clock::stopSWatch()
+{
+		swatch_totalms += swatch_start.elapsed();
+		set->setText( tr( "Start" ) );
+		t->stop();
+		swatch_running = FALSE;
+		toggleScreenSaver( TRUE );
+		updateClock();
+}
+
+
 void Clock::slotSet()
 {
   if ( t->isActive() )
 		{
-			swatch_totalms += swatch_start.elapsed();
-			set->setText( tr( "Start" ) );
-			t->stop();
-			swatch_running = FALSE;
-			toggleScreenSaver( TRUE );
-			updateClock();
+				startSWatch();
 		}
   else
 		{
-			swatch_start.start();
-			set->setText( tr( "Stop" ) );
-			t->start( 1000 );
-			swatch_running = TRUE;
-				// disable screensaver while stop watch is running
-			toggleScreenSaver( FALSE );
+				stopSWatch();
 		}
 }
 
 void Clock::slotReset()
 {
   t->stop();
   swatch_start.start();
   swatch_totalms = 0;
 
   if ( swatch_running )
     t->start( 1000 );
 
   updateClock();
 }
 
 void Clock::modeSelect( int m )
 {
 	qDebug("Clock::modeSelect( %d) ", m);
 	if ( m )
 		{
 			lcd->setNumDigits( 8 + 1 + sw_prec );
 			lcd->setMinimumWidth( lcd->sizeHint().width() );
 			set->setEnabled( TRUE );
 			reset->setEnabled( TRUE );
 			ampmLabel->hide();
 
 			if ( !swatch_running )
 				t->stop();
 		}
   else
 		{
 			lcd->setNumDigits( 5 );
 			lcd->setMinimumWidth( lcd->sizeHint().width() );
 			set->setEnabled( FALSE );
 			reset->setEnabled( FALSE );
 			t->start( 1000 );
 		}
 
 	Config config( "Clock" );
 	config.setGroup( "Mode" );
 	config.writeEntry( "clockMode", m );
   updateClock();
 }
 
 //this sets the alarm time
 void Clock::slotSetAlarm()
 {
   if ( !snoozeBtn->isHidden() )
@@ -556,80 +567,74 @@ void AnalogClock::drawPointers ( QPainter *p, const QRect &r, const QColor &c, c
     h2 = rotate( center, h2, 30 * ( t.hour() % 12 ) + t.minute() / 2 );
     p-> setPen ( QPen ( c, 3 ));
     p-> drawLine ( h1, h2 );
   }
 
   if ( !t2 || ( t. minute ( ) != t2-> minute ( ))) {
 			// draw minute pointer
     m1 = rotate( center, m1, t.minute() * 6 );
     m2 = rotate( center, m2, t.minute() * 6 );
     p-> setPen ( QPen ( c, 2 ));
     p-> drawLine ( m1, m2 );
   }
 
   if ( !t2 || ( t. second ( ) != t2-> second ( ))) {
 			// draw second pointer
     s1 = rotate( center, s1, t.second() * 6 );
     s2 = rotate( center, s2, t.second() * 6 );
     p-> setPen ( QPen ( c, 1 ));
     p-> drawLine ( s1, s2 );
   }
 }
 
 void AnalogClock::display( const QTime& t )
 {
   currTime = t;
   clear = false;
   repaint( false );
   clear = true;
 }
 
 QPoint AnalogClock::rotate( QPoint c, QPoint p, int a )
 {
   double angle = deg2rad * ( - a + 180 );
   double nx = c.x() - ( p.x() - c.x() ) * cos( angle ) -
               ( p.y() - c.y() ) * sin( angle );
   double ny = c.y() - ( p.y() - c.y() ) * cos( angle ) +
               ( p.x() - c.x() ) * sin( angle );
   return QPoint( nx, ny );
 }
 
 void Clock::slotAdjustTime()
 {
   QCopEnvelope e( "QPE/System", "execute(QString)" );
   e << QString( "systemtime" );
 }
 
 void Clock::slotStartTimer()
 {
-  Config cfg( "Clock" );
-  cfg.setGroup( "Mode" );
-	int mode = cfg.readBoolEntry( "clockMode");
   if ( clockRB->isChecked() )
 		setSwatchMode( 1);
-	slotSet();
+	startSWatch();
 }
 
 void Clock::slotStopTimer()
 {
-  Config cfg( "Clock" );
-  cfg.setGroup( "Mode" );
-	int mode = cfg.readBoolEntry( "clockMode");
   if ( clockRB->isChecked() )
 		setSwatchMode( 1);
-slotSet();
+	stopSWatch();
 }
 
 void Clock::slotResetTimer()
 {
   if ( clockRB->isChecked() )
 		setSwatchMode( 1);
 slotReset();
 }
 
 void Clock::setSwatchMode(int mode)
 {
-	qDebug("Clock::setSwatchMode( %d)"), mode;
+	qDebug("Clock::setSwatchMode( %d)", mode);
 	swatchRB->setChecked( mode);
 	clearClock( );
 	modeSelect( mode );
 }
diff --git a/noncore/tools/clock/clock.h b/noncore/tools/clock/clock.h
index 9b00e28..1e5aa2c 100644
--- a/noncore/tools/clock/clock.h
+++ b/noncore/tools/clock/clock.h
@@ -55,52 +55,54 @@ private:
 	void drawPointers ( QPainter *, const QRect &r, const QColor &c, const QTime &t, const QTime *t2 = 0 );
 
 };
 
 class Clock : public QVBox
 {
     Q_OBJECT
 
 public:
     Clock( QWidget * parent = 0, const char * name = 0, WFlags f=0 );
     ~Clock();
     QDateTime when;
     bool bSound;
     int hour, minute, snoozeTime;
 private slots:
     void slotSet();
     void slotReset();
     void modeSelect(int);
     void updateClock();
     void changeClock( bool );
     void slotSetAlarm();
     void slotSnooze();
     void slotToggleAlarm();
     void alarmOn();
     void alarmOff();
     void appMessage(const QCString& msg, const QByteArray& data);
     void timerEvent( QTimerEvent *e );
     void slotAdjustTime();
 
 	void slotStartTimer();
 	void slotStopTimer();
 	void slotResetTimer();
 	void setSwatchMode( int );
 private:
     bool alarmBool;
     QTimer *t;
     QLCDNumber *lcd;
     QLabel *date;
     QLabel *ampmLabel;
     QPushButton *set, *reset, *alarmBtn, *snoozeBtn, *alarmOffBtn;
     QRadioButton *clockRB, *swatchRB;
     AnalogClock *aclock;
     QTime swatch_start;
     int swatch_totalms;
     bool swatch_running;
     bool ampm;
     void clearClock();
     void clearTimer();
+    void startSWatch();
+    void stopSWatch();
 };
 
 #endif