[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdeedu/kstars/kstars/tools
From:       Alexey Khudyakov <alexey.skladnoy () gmail ! com>
Date:       2011-02-25 12:26:57
Message-ID: 20110225122657.52484AC8C2 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1222704 by khudyakov:

Switch to for loop.

Maybe it's a bit too big

 M  +5 -4      skycalendar.cpp  


--- trunk/KDE/kdeedu/kstars/kstars/tools/skycalendar.cpp #1222703:1222704
@@ -140,11 +140,14 @@
 void SkyCalendar::addPlanetEvents( int nPlanet ) {
     KSPlanetBase *ksp = KStarsData::Instance()->skyComposite()->planet( nPlanet );
     int y = scUI->Year->value();
-    KStarsDateTime kdt( QDate( y, 1, 1 ), QTime( 12, 0, 0 ) );
     QColor pColor = KSPlanetBase::planetColor[nPlanet];
 
     QVector<QPointF> vRise, vSet, vTransit;
-    while( kdt.date().year() == y ) {
+    
+    for( KStarsDateTime kdt( QDate( y, 1, 1 ), QTime( 12, 0, 0 ) );
+         kdt.date().year() == y;
+         kdt = kdt.addDays( 7 ))
+    {
         float dy = float( kdt.date().daysInYear() - kdt.date().dayOfYear() );
 
         //Compute rise/set/transit times.  If they occur before noon, 
@@ -165,8 +168,6 @@
         vRise    << QPointF( timeToHours( rtime ), dy );
         vSet     << QPointF( timeToHours( stime ), dy );
         vTransit << QPointF( timeToHours( ttime ), dy );
-
-        kdt = kdt.addDays( 7 );
     }
 
     //Now, find continuous segments in each QVector and add each segment 
[prev in list] [next in list] [prev in thread] [next in thread]