From: aclement <aclement>
Date: Fri, 3 Oct 2008 23:30:39 +0000 (+0000)
Subject: make cleanup part of compiler discard
X-Git-Tag: V1_6_2~4
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e78ca8b57dbb82776f96d288353f659b2c3c3cfd;p=aspectj.git

make cleanup part of compiler discard
---

diff --git a/ajde.core/src/org/aspectj/ajde/core/AjCompiler.java b/ajde.core/src/org/aspectj/ajde/core/AjCompiler.java
index 758073d83..72eb4058c 100644
--- a/ajde.core/src/org/aspectj/ajde/core/AjCompiler.java
+++ b/ajde.core/src/org/aspectj/ajde/core/AjCompiler.java
@@ -88,13 +88,6 @@ public class AjCompiler {
 		}
 	}
 
-	/**
-	 * Cleanup the compiler instance, ready for removal.
-	 */
-	public void cleanup() {
-		buildManager.cleanup();
-	}
-
 	/**
 	 * Perform a full build.
 	 */
@@ -106,10 +99,12 @@ public class AjCompiler {
 
 	/**
 	 * Clear the incremental state associated with this AjCompiler from the IncrementalStateManager. This is necessary until AjState
-	 * is reworked and there's an AjState associated with an AjCompiler rather than requiring a map of them.
+	 * is reworked and there's an AjState associated with an AjCompiler rather than requiring a map of them. If the environment is
+	 * not cleaned up then jar locks may be kept.
 	 */
 	public void clearLastState() {
 		IncrementalStateManager.removeIncrementalStateInformationFor(compilerId);
+		buildManager.cleanupEnvironment();
 	}
 
 	/**
diff --git a/ajde.core/src/org/aspectj/ajde/core/internal/AjdeCoreBuildManager.java b/ajde.core/src/org/aspectj/ajde/core/internal/AjdeCoreBuildManager.java
index c22248f46..074222b51 100644
--- a/ajde.core/src/org/aspectj/ajde/core/internal/AjdeCoreBuildManager.java
+++ b/ajde.core/src/org/aspectj/ajde/core/internal/AjdeCoreBuildManager.java
@@ -391,7 +391,7 @@ public class AjdeCoreBuildManager {
 		return ajBuildManager.getCustomMungerFactory();
 	}
 
-	public void cleanup() {
-		ajBuildManager.cleanup();
+	public void cleanupEnvironment() {
+		ajBuildManager.cleanupEnvironment();
 	}
 }