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

List:       kde-commits
Subject:    KDE/kdeedu/kstars/kstars/skyobjects
From:       Alexey Khudyakov <alexey.skladnoy () gmail ! com>
Date:       2009-11-27 21:01:10
Message-ID: 1259355670.366454.26493.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 1055331 by khudyakov:

Do not create empty SkyPoint and assign another Skypoint to it
later. Use constructor instead

 M  +1 -2      skypoint.cpp  


--- trunk/KDE/kdeedu/kstars/kstars/skyobjects/skypoint.cpp #1055330:1055331
@@ -261,14 +261,13 @@
 
     static KSSun* sun;
     const dms maxAngle( 1.75 * ( 30.0 / 200.0) / dms::DegToRad );
-    SkyPoint sp;
 
     if( !sun ) // Hope we don't destroy the sun before the program ends!
         sun = (KSSun*) KStarsData::Instance()->skyComposite()->findByName( "Sun" );
     //    kDebug() << "bendlight says maximum correctable angle is " << maxAngle.Degrees();
 
     // TODO: We can make this code more efficient
-    sp = SkyPoint( sun->ra0(), sun->dec0() );
+    SkyPoint sp( sun->ra0(), sun->dec0() );
     //    kDebug() << "The unaberrated sun is " << sp.angularDistanceTo( sun ).Degrees() << "deg away";
     if( fabs( angularDistanceTo( &sp ).radians() ) <= maxAngle.radians() ) {
         // We correct for GR effects
[prev in list] [next in list] [prev in thread] [next in thread]