[jboss-svn-commits] JBL Code SVN: r5905 - labs/jbossrules/branches/3.0.x/drools-ide/src/main/java/org/drools/ide/menu

[jboss-svn-commits at lists.jboss.org]

Author: KrisVerlaenen
Date: 2006-08-15 18:13:08 -0400 (Tue, 15 Aug 2006)
New Revision: 5905

Modified:
   labs/jbossrules/branches/3.0.x/drools-ide/src/main/java/org/drools/ide/menu/RuleHelperActionDelegate.java
Log:
JBRULES-426: Memory leak in RuleHelperActionDelegate
 - existing menu is disposed off when setting a new one

Modified: labs/jbossrules/branches/3.0.x/drools-ide/src/main/java/org/drools/ide/menu/RuleHelperActionDelegate.java
===================================================================
--- labs/jbossrules/branches/3.0.x/drools-ide/src/main/java/org/drools/ide/menu/RuleHelperActionDelegate.java	2006-08-15 22:04:20 UTC (rev 5904)
+++ labs/jbossrules/branches/3.0.x/drools-ide/src/main/java/org/drools/ide/menu/RuleHelperActionDelegate.java	2006-08-15 22:13:08 UTC (rev 5905)
@@ -36,11 +36,12 @@
     IWorkbenchWindowPulldownDelegate {
 
     private IWorkbench workbench;
+    private Menu menu;
 
     /** Return a menu which launches the various wizards */
     public Menu getMenu(Control parent) {
 
-        Menu menu = new Menu( parent );
+        setMenu( new Menu( parent ) );
         
         final Shell shell = parent.getShell();
         addProjectWizard( menu,
@@ -58,6 +59,13 @@
         return menu;
     }
 
+    private void setMenu(Menu menu) {
+        if (this.menu != null) {
+            this.menu.dispose();
+        }
+        this.menu = menu;
+    }
+
     private void addDTWizard(Menu menu,
                              final Shell shell) {
         MenuItem dsl = new MenuItem( menu,