Remove ssg kids in a an attepmpt to fix one memory leak. We think the

[flightgear.git] / src / Scenery / tileentry.cxx

diff --git a/src/Scenery/tileentry.cxx b/src/Scenery/tileentry.cxx
index d90f465f263cb0e6a4926d707155d87a9d81c69d..7df08fa7419ac65823f980a1567b3f314edd9214 100644 (file)
--- a/src/Scenery/tileentry.cxx
+++ b/src/Scenery/tileentry.cxx
@@ -42,7 +42,10 @@ FG_USING_STD(mem_fun_ref);
 // Constructor
 FGTileEntry::FGTileEntry ( void )
     : ncount(0),
-      state(Unused)
+      state(Unused),
+      vtlist(NULL),
+      vnlist(NULL),
+      tclist(NULL)
 {
     nodes.clear();
 }
@@ -55,17 +58,96 @@ FGTileEntry::~FGTileEntry ( void ) {
 }
 
 
-// Step through the fragment list, deleting the display list, then
-// the fragment, until the list is empty.
+// Step through the fragment list, deleting the display list, then the
+// fragment, until the list is empty.  Also delete the arrays used by
+// ssg as well as the whole ssg branch
 void
-FGTileEntry::release_fragments()
+FGTileEntry::free_tile()
 {
     FG_LOG( FG_TERRAIN, FG_DEBUG,
            "FREEING TILE = (" << tile_bucket << ")" );
-    for_each( begin(), end(),
-             mem_fun_ref( &fgFRAGMENT::deleteDisplayList ));
-    fragment_list.erase( begin(), end() );
+
+    // mark tile unused
     mark_unused();
+
+    // delete fragment list and node list
+    FG_LOG( FG_TERRAIN, FG_DEBUG,
+           "  deleting " << fragment_list.size() << " fragments" );
+    fragment_list.clear();
+    FG_LOG( FG_TERRAIN, FG_DEBUG,
+           "  deleting " << nodes.size() << " nodes" );
+    nodes.clear();
+
+    // delete the ssg structures
+    FG_LOG( FG_TERRAIN, FG_DEBUG,
+           "  deleting vertex, normal, and texture coordinate arrays" );
+    FG_LOG( FG_TERRAIN, FG_DEBUG,
+           "    deleting vertex array" );
+    if ( vtlist != NULL ) {
+       delete vtlist;
+    }
+    FG_LOG( FG_TERRAIN, FG_DEBUG,
+           "    deleting normal array" );
+    if ( vnlist != NULL ) {
+       delete vnlist;
+    }
+    FG_LOG( FG_TERRAIN, FG_DEBUG,
+           "    deleting texture coordinate array" );
+    if ( tclist != NULL ) {
+       delete tclist;
+    }
+
+    // delete the ssg branch
+
+    // make sure we have a sane number of parents
+    int pcount = select_ptr->getNumParents();
+    if ( pcount > 0 ) {
+       // find the first parent (should only be one)
+       ssgBranch *parent = select_ptr->getParent( 0 ) ;
+       // find the number of kids this parent has
+       int kcount = parent->getNumKids();
+       // find the kid that matches our original select_ptr
+       bool found_kid = false;
+       for ( int i = 0; i < kcount; ++i ) {
+           ssgEntity *kid = parent->getKid( i );
+           if ( kid == select_ptr ) {
+               FG_LOG( FG_TERRAIN, FG_DEBUG,
+                       "Found a kid to delete " << kid);
+               found_kid = true;
+               parent->removeKid( i );
+           }
+       }
+       if ( ! found_kid ) {
+           FG_LOG( FG_TERRAIN, FG_ALERT,
+                   "Couldn't find the kid to delete!  Dying" );
+           exit(-1);
+       }
+    } else {
+       FG_LOG( FG_TERRAIN, FG_ALERT,
+               "Parent count is zero for an ssg tile!  Dying" );
+       exit(-1);
+    }
 }
 
 
+// when a tile is still in the cache, but not in the immediate draw
+// list, it can still remain in the scene graph, but we use a range
+// selector to disable it from ever being drawn.
+void 
+FGTileEntry::ssg_disable() {
+    // cout << "TILE STATE = " << state << endl;
+    if ( state == Scheduled_for_use ) {
+       state = Scheduled_for_cache;
+    } else if ( state == Scheduled_for_cache ) {
+       // do nothing
+    } else if ( (state == Loaded) || (state == Cached) ) {
+       state = Cached;
+       // cout << "DISABLING SSG NODE" << endl;
+       select_ptr->select(0);
+    } else {
+       FG_LOG( FG_TERRAIN, FG_ALERT,
+               "Trying to disable an unused tile!  Dying" );
+       exit(-1);
+    }  
+    // cout << "TILE STATE = " << state << endl;
+}