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

List:       kde-commits
Subject:    KDE/kdeedu/kstars/kstars/skycomponents
From:       Jason Harris <kstars () 30doradus ! org>
Date:       2005-09-17 2:48:05
Message-ID: 1126925285.255977.7710.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 461267 by harris:

Added EclipticComponent.  Modified destructor of 
SolarSystemComposite (can you review this change, Thomas?)

CCMAIL: kstars-devel@kde.org


 M  +1 -1      Makefile.am  
 A             eclipticcomponent.cpp   [License: GPL (v2+)]
 A             eclipticcomponent.h   [License: GPL (v2+)]
 M  +2 -1      equatorcomponent.cpp  
 M  +3 -0      solarsystemcomposite.cpp  


--- trunk/KDE/kdeedu/kstars/kstars/skycomponents/Makefile.am #461266:461267
@@ -5,7 +5,7 @@
 			constellationboundarycomponent.cpp \
 			coordinategridcomposite.cpp coordinategridcomponent.cpp \
 			customcatalogscomponent.cpp deepskycomponent.cpp \
-			equatorcomponent.cpp horizoncomponent.cpp \
+			eclipticcomponent.cpp equatorcomponent.cpp horizoncomponent.cpp \
 			jupitermoonscomponent.cpp milkywaycomponent.cpp \
 			mooncomponent.cpp skycomponent.cpp skycomposite.cpp \
 			skymapcomposite.cpp solarsystemcomposite.cpp \
--- trunk/KDE/kdeedu/kstars/kstars/skycomponents/equatorcomponent.cpp #461266:461267
@@ -35,6 +35,7 @@
 
 EquatorComponent::~EquatorComponent()
 {
+  while ( ! Equator.isEmpty() ) delete Equator.takeFirst();
 }
 
 // was KStarsData::initGuides(KSNumbers *num)
@@ -56,7 +57,7 @@
 void EquatorComponent::update(KStarsData *data, KSNumbers *num, bool needNewCoords)
 {
 	if ( Options::showEquator() ) {
-		for ( SkyPoint *p = Equator.first(); p; p = Equator.next() ) {
+	  foreach ( SkyPoint *p, Equator ) {
 			p->EquatorialToHorizontal( LST, data->geo()->lat() );
 		}
 	}
--- trunk/KDE/kdeedu/kstars/kstars/skycomponents/solarsystemcomposite.cpp #461266:461267
@@ -35,6 +35,9 @@
 
 SolarSystemComposite::~SolarSystemComposite::()
 {
+  //JH: is this necessary to call, or is it done automatically?
+  SkyComposite::~SkyComposite();
+
 	delete Earth;
 	delete Sun;
 	delete Venus;
[prev in list] [next in list] [prev in thread] [next in thread]