summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAkarsh Simha <akarsh@kde.org>2016-10-14 06:32:07 (GMT)
committerAkarsh Simha <akarsh@kde.org>2016-10-14 10:28:36 (GMT)
commite108f42e4467070221ff33a83695349cbdb8328f (patch)
tree3ea352f5e1ecc11f23bbeccfd0ede6c15edbe4c6
parentc1247ca1c7c230eb63886c3488a0f99900fe9cad (diff)
More FIXMEs
-rw-r--r--kstars/skycomponents/skymapcomposite.cpp13
1 files changed, 12 insertions, 1 deletions
diff --git a/kstars/skycomponents/skymapcomposite.cpp b/kstars/skycomponents/skymapcomposite.cpp
index 1627981..3b2395a 100644
--- a/kstars/skycomponents/skymapcomposite.cpp
+++ b/kstars/skycomponents/skymapcomposite.cpp
@@ -659,7 +659,11 @@ void SkyMapComposite::reloadDeepSky() {
current_map->setClickedPoint( current_map->clickedObject() );
current_map->slotCenter();
- //Remove and Regenerate set of catalog objects
+ // Remove and Regenerate set of catalog objects
+ //
+ // FIXME: Why should we do this? Because it messes up observing
+ // list really bad to delete and regenerate SkyObjects.
+
SkyMapDrawAbstract::setDrawLock(true);
delete m_CustomCatalogs;
m_CustomCatalogs = new SkyComposite( this );
@@ -673,6 +677,13 @@ void SkyMapComposite::reloadDeepSky() {
continue;
m_CustomCatalogs->addComponent( new CatalogComponent( this, allcatalogs.at(i), false, i ), 5 ); // FIXME: Should this be 6 or 5? See SkyMapComposite::SkyMapComposite()
}
+
+ // FIXME: We should also reload anything that refers to SkyObject
+ // * in memory, because all the old SkyObjects are now gone! This
+ // includes the observing list. Otherwise, expect a bad, bad crash
+ // that is hard to debug! -- AS
+
+
SkyMapDrawAbstract::setDrawLock(false);
#endif
}