2 files changed, 59 insertions, 54 deletions

diff --git a/noncore/tools/clock/analogclock.cpp b/noncore/tools/clock/analogclock.cpp
index 7f8ca67..c5f0155 100644
--- a/noncore/tools/clock/analogclock.cpp
+++ b/noncore/tools/clock/analogclock.cpp
@@ -40,83 +40,88 @@ 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();
 
     if ( r.width() < r.height() ) {
         r.setY( (r.height() - r.width())/2 );
         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 );
 
-    QBrush bgBrush = colorGroup().brush( QColorGroup::Background );
-
-//    if ( clear ){
-        p->fillRect ( r, bgBrush );
-
-
-        // 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 );
-//    }
+    const int w_tick = r.width()/300+1;
+    const int w_sec = r.width()/400+1;
+    const int w_hour = r.width()/80+1;
 
-    QColor col = colorGroup().color( QColorGroup::Text );
-    p->setBrush( col );
-    drawPointers ( p, r, col, currTime );
-
-
-    prevTime = currTime;
-}
-
-void AnalogClock::drawPointers ( QPainter *p, const QRect &r, const QColor &c, const QTime &t, const QTime *t2 )
-{
-    QPoint center = r. center ( );
+    QPoint l1( r.x() + r.width() / 2, r.y() + 2 );
+    QPoint l2( r.x() + r.width() / 2, r.y() + 8 );
 
+    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 h1( center. x ( ), r. y ( ) + r. height ( ) / 4 );
-    QPoint h2( center. x ( ), center. y ( ) );
+    QPoint m1( r.x() + r.width() / 2, r.y() + r.height() / 9 );
+    QPoint m2( r.x() + r.width() / 2, r.y() + r.height() / 2 );
 
-    QPoint m1( center. x ( ), r.y() + r.height() / 8 );
-    QPoint m2( center. x ( ), center. y ( ) );
+    QPoint s1( r.x() + r.width() / 2, r.y() + 8 );
+    QPoint s2( r.x() + r.width() / 2, r.y() + r.height() / 2 );
 
-    QPoint s1( center. x ( ), r. y ( ) + 8 );
-    QPoint s2( center. x ( ), center. y ( ) );
+    QColor color( clear ? backgroundColor() : black );
+    QTime time = clear ? prevTime : currTime;
 
+    if ( clear && prevTime.secsTo(currTime) > 1 ) {
+        p->eraseRect( rect() );
+        return;
+    }
 
-    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 ( !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 ) );
     }
 
-    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 ( !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 ( !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 );
+    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 );
     }
-}
 
+    // 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 );
+
+    // cap
+    p->setBrush(color);
+    p->drawEllipse( center.x()-w_hour/2, center.y()-w_hour/2, w_hour, w_hour );
+
+    if ( !clear )
+        prevTime = currTime;
+}
 
 // Dijkstra's bisection algorithm to find the square root as an integer.
 
 static uint int_sqrt(uint n)
@@ -180,11 +185,12 @@ 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 5f30d68..3aa035e 100644
--- a/noncore/tools/clock/analogclock.h
+++ b/noncore/tools/clock/analogclock.h
@@ -35,9 +35,8 @@ 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 );