From 294bddc32c1214231c40c7e9406aea412bb4a5a4 Mon Sep 17 00:00:00 2001
From: Sean Kasun <skasun@gmail.com>
Date: Mon, 18 Mar 2013 16:44:34 -0700
Subject: [PATCH] cleaned up memory freeing at close

---
 biomeidentifier.cpp   |  8 ++++++++
 biomeidentifier.h     |  1 +
 blockidentifier.cpp   | 10 ++++++++++
 blockidentifier.h     |  1 +
 definitionmanager.cpp |  7 +++++++
 definitionmanager.h   |  1 +
 dimensions.cpp        |  5 +++++
 7 files changed, 33 insertions(+)

diff --git a/biomeidentifier.cpp b/biomeidentifier.cpp
index f03b808b..0cfb073b 100644
--- a/biomeidentifier.cpp
+++ b/biomeidentifier.cpp
@@ -33,6 +33,14 @@ BiomeIdentifier::BiomeIdentifier()
 {
 	unknownBiome.name="Unknown";
 }
+BiomeIdentifier::~BiomeIdentifier()
+{
+	for (int i=0;i<packs.length();i++)
+	{
+		for (int j=0;j<packs[i].length();j++)
+			delete packs[i][j];
+	}
+}
 
 BiomeInfo &BiomeIdentifier::getBiome(int biome)
 {
diff --git a/biomeidentifier.h b/biomeidentifier.h
index 7efc555a..9a63dd23 100644
--- a/biomeidentifier.h
+++ b/biomeidentifier.h
@@ -44,6 +44,7 @@ class BiomeIdentifier
 {
 public:
 	BiomeIdentifier();
+	~BiomeIdentifier();
 	int addDefinitions(JSONArray *,int pack=-1);
 	void enableDefinitions(int);
 	void disableDefinitions(int);
diff --git a/blockidentifier.cpp b/blockidentifier.cpp
index d7d55f6c..3382af39 100644
--- a/blockidentifier.cpp
+++ b/blockidentifier.cpp
@@ -38,6 +38,16 @@ BlockIdentifier::BlockIdentifier()
 	unknownBlock.flags=0;
 	unknownBlock.name="Unknown";
 }
+BlockIdentifier::~BlockIdentifier()
+{
+	for (int i=0;i<65536;i++)
+		cache[i]=NULL;
+	for (int i=0;i<packs.length();i++)
+	{
+		for (int j=0;j<packs[i].length();j++)
+			delete packs[i][j];
+	}
+}
 
 // this routine is ridiculously slow
 BlockInfo &BlockIdentifier::getBlock(int id, int data)
diff --git a/blockidentifier.h b/blockidentifier.h
index abcefad0..5a05f553 100644
--- a/blockidentifier.h
+++ b/blockidentifier.h
@@ -61,6 +61,7 @@ class BlockIdentifier
 {
 public:
 	BlockIdentifier();
+	~BlockIdentifier();
 	int addDefinitions(JSONArray *,int pack=-1);
 	void enableDefinitions(int);
 	void disableDefinitions(int);
diff --git a/definitionmanager.cpp b/definitionmanager.cpp
index fc485af0..56b1cb98 100644
--- a/definitionmanager.cpp
+++ b/definitionmanager.cpp
@@ -117,6 +117,13 @@ DefinitionManager::DefinitionManager(QWidget *parent) : QWidget(parent)
 	refresh();
 }
 
+DefinitionManager::~DefinitionManager()
+{
+    delete dimensionList;
+    delete blocks;
+    delete biomes;
+}
+
 BlockIdentifier *DefinitionManager::blockIdentifier()
 {
 	return blocks;
diff --git a/definitionmanager.h b/definitionmanager.h
index d6dc4449..4ed3d374 100644
--- a/definitionmanager.h
+++ b/definitionmanager.h
@@ -64,6 +64,7 @@ class DefinitionManager : public QWidget
 	Q_OBJECT
 public:
 	explicit DefinitionManager(QWidget *parent = 0);
+    ~DefinitionManager();
 	void attachMapView(MapView *mapview);
 	QSize minimumSizeHint() const;
 	QSize sizeHint() const;
diff --git a/dimensions.cpp b/dimensions.cpp
index fec45206..c385f6a6 100644
--- a/dimensions.cpp
+++ b/dimensions.cpp
@@ -48,6 +48,11 @@ Dimensions::Dimensions()
 }
 Dimensions::~Dimensions()
 {
+	for (int i=0;i<packs.length();i++)
+	{
+		for (int j=0;j<packs[i].length();j++)
+			delete packs[i][j];
+	}
 }
 
 void Dimensions::enableDefinitions(int pack)