06a9e368b9b085df0c78c5391decbd1ced9da8b6
tdreszer
  Thu Jul 28 16:23:17 2011 -0700
Cleaned up the tdbExtras a little.  Jim recommended always allocating.  Currently leaving in an assert, but will remove it on next check-in
diff --git src/hg/lib/trackDb.c src/hg/lib/trackDb.c
index d95a7ba..2cde3b9 100644
--- src/hg/lib/trackDb.c
+++ src/hg/lib/trackDb.c
@@ -159,31 +159,31 @@
 if ((el = *pEl) == NULL) return;
 freeMem(el->track);
 freeMem(el->shortLabel);
 freeMem(el->type);
 freeMem(el->longLabel);
 /* All strings in restrictList are allocated at once, so only need to free first. */
 if (el->restrictList != NULL)
     freeMem(el->restrictList[0]);
 freeMem(el->restrictList);
 freeMem(el->url);
 freeMem(el->html);
 freeMem(el->grp);
 freeMem(el->settings);
 hashFree(&el->settingsHash);
 hashFree(&el->overrides);
-freeMem(&el->tdbExtras); // For now leak contents, because complex structs would also leak
+tdbExtrasFree(&el->tdbExtras);
 freez(pEl);
 // #endif /* NOT */
 }
 
 void trackDbFreeList(struct trackDb **pList)
 /* Free a list of dynamically allocated trackDb's */
 {
 return;
 // #ifdef NOT
 struct trackDb *el, *next;
 
 for (el = *pList; el != NULL; el = next)
     {
     next = el->next;
     trackDbFree(&el);