fix variable editor memory leak

akeshet · Sep 16, 2012 · 21fab32 · 21fab32
1 parent 20982ac
commit 21fab32
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/WordGenerator/Controls/VariablesTab/VariableEditor.Designer.cs b/WordGenerator/Controls/VariablesTab/VariableEditor.Designer.cs
diff --git a/WordGenerator/Controls/VariablesTab/VariableEditor.cs b/WordGenerator/Controls/VariablesTab/VariableEditor.cs
@@ -209,7 +209,7 @@ private void attemptDeleteVariable()
             }
         }
 
-        private void button1_Click(object sender, EventArgs e)
+        private void deleteVariableButtonClick(object sender, EventArgs e)
         {
             attemptDeleteVariable();
         }

diff --git a/WordGenerator/Controls/VariablesTab/VariablesAndListPage.cs b/WordGenerator/Controls/VariablesTab/VariablesAndListPage.cs
@@ -141,6 +141,7 @@ private void layoutVariables()
                     for (int i = 0; i < extras; i++)
                     {
                         variablesPanel.Controls.Remove(variableEditors[0]);
+                        variableEditors[0].Dispose();
                         variableEditors.RemoveAt(0);
                     }
                 }
@@ -226,6 +227,7 @@ void ved_variableDeleted(object sender, EventArgs e)
                 // if we know which specific editor it was
                 variableEditors.Remove(ved);
                 variablesPanel.Controls.Remove(ved);
+                ved.Dispose();
 
             }
             else