[Kstars-devel] kdeedu/kstars/kstars

[Jason Harris]

CVS commit by harris: 

Applying patch from Aaron Seigo to remove a compiler warning and clean 
up API a bit.  Will backport to 3_3_BRANCH.

Thanks, Aaron!

CCMAIL: aseigo at kde.org
CCMAIL: kstars-devel at kde.org


  M +2 -1      skyobject.cpp   1.47
  M +1 -1      skypoint.cpp   1.30
  M +3 -3      skypoint.h   1.23


--- kdeedu/kstars/kstars/skypoint.h  #1.22:1.23
@@ -51,5 +51,5 @@ public:
         *@param d Declination
         */
-        SkyPoint( dms r, dms d ) { set( r, d ); }
+        SkyPoint( const dms& r, const dms& d ) { set( r, d ); }
 
 /**Alternate constructor using pointer arguments, for convenience.
@@ -81,5 +81,5 @@ public:
         *@param d Declination
         */
-        void set( dms r, dms d );
+        void set( const dms& r, const dms& d );
 
 /**Overloaded member function, provided for convenience.
@@ -88,5 +88,5 @@ public:
         *@param d Declination
         */
-        void set( const dms *r, const dms *d ) { set( dms(*r), dms(*d) ); }
+        void set( const dms *r, const dms *d ) { set( *r, *d ); }
 
 /**Overloaded member function, provided for convenience.

--- kdeedu/kstars/kstars/skypoint.cpp  #1.29:1.30
@@ -23,5 +23,5 @@
 #include "ksnumbers.h"
 
-void SkyPoint::set( dms r, dms d ) {
+void SkyPoint::set( const dms& r, const dms& d ) {
         RA0.set( r );
         Dec0.set( d );

--- kdeedu/kstars/kstars/skyobject.cpp  #1.46:1.47
@@ -98,5 +98,6 @@ QTime SkyObject::riseSetTime( const KSta
         KStarsDateTime dt2 = dt;
         SkyPoint p = recomputeCoords( dt, geo );
-        p.EquatorialToHorizontal( &(geo->GSTtoLST( dt.gst() )), geo->lat() );
+        dms d( geo->GSTtoLST( dt.gst() ) );
+        p.EquatorialToHorizontal( &d, geo->lat() );
         if ( p.alt()->Degrees() < 0.0 ) {
                 if ( p.az()->Degrees() < 180.0 ) { //object has not risen yet