2 files changed, 54 insertions, 59 deletions

diff --git a/noncore/tools/clock/analogclock.cpp b/noncore/tools/clock/analogclock.cpp
index c5f0155..7f8ca67 100644
--- a/noncore/tools/clock/analogclock.cpp
+++ b/noncore/tools/clock/analogclock.cpp
@@ -41,13 +41,6 @@ QSizePolicy AnalogClock::sizePolicy() const
 
 void AnalogClock::drawContents( QPainter *p )
 {
-#if !defined(NO_DEBUG)
-    static bool first = true;
-    if ( first ) {
-        //QTOPIA_PROFILE("first paint event");
-        first = false;
-    }
-#endif
 
     QRect r = contentsRect();
 
@@ -56,72 +49,74 @@ void AnalogClock::drawContents( QPainter *p )
         r.setHeight( r.width() );
     }
 
+    QPoint l1( r.x() + r.width() / 2, r.y() + 2 );
+    QPoint l2( r.x() + r.width() / 2, r.y() + 8 );
     QPoint center( r.x() + r.width() / 2, r.y() + r.height() / 2 );
 
-    const int w_tick = r.width()/300+1;
-    const int w_sec = r.width()/400+1;
-    const int w_hour = r.width()/80+1;
+    QBrush bgBrush = colorGroup().brush( QColorGroup::Background );
 
-    QPoint l1( r.x() + r.width() / 2, r.y() + 2 );
-    QPoint l2( r.x() + r.width() / 2, r.y() + 8 );
+//    if ( clear ){
+        p->fillRect ( r, bgBrush );
 
-    QPoint h1( r.x() + r.width() / 2, r.y() + r.height() / 4 );
-    QPoint h2( r.x() + r.width() / 2, r.y() + r.height() / 2 );
 
-    QPoint m1( r.x() + r.width() / 2, r.y() + r.height() / 9 );
-    QPoint m2( r.x() + r.width() / 2, r.y() + r.height() / 2 );
+        // draw ticks
+        p->setPen( QPen( colorGroup ( ). color ( QColorGroup::Text ), 1 ) );
+        for ( int i = 0; i < 12; i++ )
+            p->drawLine( rotate( center, l1, i * 30 ), rotate( center, l2, i * 30 ) );
+//    }else {
+//        p->setBrush( bgBrush );
+//        drawPointers ( p, r, colorGroup ( ). color ( QColorGroup::Background ), prevTime, &currTime );
+//    }
 
-    QPoint s1( r.x() + r.width() / 2, r.y() + 8 );
-    QPoint s2( r.x() + r.width() / 2, r.y() + r.height() / 2 );
+    QColor col = colorGroup().color( QColorGroup::Text );
+    p->setBrush( col );
+    drawPointers ( p, r, col, currTime );
 
-    QColor color( clear ? backgroundColor() : black );
-    QTime time = clear ? prevTime : currTime;
 
-    if ( clear && prevTime.secsTo(currTime) > 1 ) {
-        p->eraseRect( rect() );
-        return;
-    }
+    prevTime = currTime;
+}
 
-    if ( !clear ) {
-        // draw ticks
-        p->setPen( QPen( color, w_tick ) );
-        for ( int i = 0; i < 12; i++ )
-            p->drawLine( rotate( center, l1, i * 30 ), rotate( center, l2, i * 30 ) );
-    }
+void AnalogClock::drawPointers ( QPainter *p, const QRect &r, const QColor &c, const QTime &t, const QTime *t2 )
+{
+    QPoint center = r. center ( );
 
-    if ( !clear || prevTime.minute() != currTime.minute() ||
-            prevTime.hour() != currTime.hour() ) {
-        // draw hour pointer
-        h1 = rotate( center, h1, 30 * ( time.hour() % 12 ) + time.minute() / 2 );
-        h2 = rotate( center, h2, 30 * ( time.hour() % 12 ) + time.minute() / 2 );
-        p->setPen( color );
-        p->setBrush( color );
-        drawHand( p, h1, h2 );
-    }
 
-    if ( !clear || prevTime.minute() != currTime.minute() ) {
-        // draw minute pointer
-        m1 = rotate( center, m1, time.minute() * 6 );
-        m2 = rotate( center, m2, time.minute() * 6 );
-        p->setPen( color );
-        p->setBrush( color );
-        drawHand( p, m1, m2 );
-    }
+    QPoint h1( center. x ( ), r. y ( ) + r. height ( ) / 4 );
+    QPoint h2( center. x ( ), center. y ( ) );
 
-    // draw second pointer
-    s1 = rotate( center, s1, time.second() * 6 );
-    s2 = rotate( center, s2, time.second() * 6 );
-    p->setPen( QPen( color, w_sec ) );
-    p->drawLine( s1, s2 );
+    QPoint m1( center. x ( ), r.y() + r.height() / 8 );
+    QPoint m2( center. x ( ), center. y ( ) );
 
-    // cap
-    p->setBrush(color);
-    p->drawEllipse( center.x()-w_hour/2, center.y()-w_hour/2, w_hour, w_hour );
+    QPoint s1( center. x ( ), r. y ( ) + 8 );
+    QPoint s2( center. x ( ), center. y ( ) );
 
-    if ( !clear )
-        prevTime = currTime;
+
+    if ( !t2 || ( t. minute ( ) != t2-> minute ( ) || t. hour ( ) != t2-> hour ( ))) {
+        // draw hour pointer
+        h1 = rotate( center, h1, 30 * ( t.hour() % 12 ) + t.minute() / 2 );
+        h2 = rotate( center, h2, 30 * ( t.hour() % 12 ) + t.minute() / 2 );
+        p-> setPen ( QPen ( c, 3 ));
+        drawHand( p, 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 ));
+        drawHand( p, 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 );
+    }
 }
 
+
 // Dijkstra's bisection algorithm to find the square root as an integer.
 
 static uint int_sqrt(uint n)
@@ -186,10 +181,9 @@ void AnalogClock::drawHand( QPainter *p, QPoint p1, QPoint p2 )
 void AnalogClock::display( const QTime& t )
 {
     currTime = t;
-    clear = true;
-    repaint( false );
     clear = false;
     repaint( false );
+    clear = true;
 }
 
 QPoint AnalogClock::rotate( QPoint c, QPoint p, int a )
diff --git a/noncore/tools/clock/analogclock.h b/noncore/tools/clock/analogclock.h
index 3aa035e..5f30d68 100644
--- a/noncore/tools/clock/analogclock.h
+++ b/noncore/tools/clock/analogclock.h
@@ -36,6 +36,7 @@ public:
 protected:
     void drawContents( QPainter *p );
     void drawHand( QPainter *p, QPoint, QPoint );
+    void drawPointers ( QPainter *p, const QRect &r, const QColor &c, const QTime &t, const QTime *t2 = 0);
 
 private:
     QPoint rotate( QPoint center, QPoint p, int angle );