Author: akazakov Date: 2011-09-01 16:30:48 -0400 (Thu, 01 Sep 2011) New Revision: 34492 Modified: trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/ValidatorManager.java Log: https://issues.jboss.org/browse/JBIDE-9637 On workspaces with complex dependencies between CDI projects, CDI validation self-disables after full build. Modified: trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/ValidatorManager.java =================================================================== --- trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/ValidatorManager.java 2011-09-01 20:19:10 UTC (rev 34491) +++ trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/ValidatorManager.java 2011-09-01 20:30:48 UTC (rev 34492) @@ -63,21 +63,16 @@ return OK_STATUS; } IValidationContextManager validationContextManager = validationHelper.getValidationContextManager(); - Set<IProject> allRootProjects = validationContextManager.getRootProjects(); - Set<IProject> rootProjects = new HashSet<IProject>(); + Set<IProject> rootProjects = validationContextManager.getRootProjects(); IStatus status = OK_STATUS; synchronized (validatingProjects) { - for (IProject rootProject : allRootProjects) { - if(!validatingProjects.contains(rootProject)) { - // Validate root projects that is not being validated yet. - rootProjects.add(rootProject); - validatingProjects.add(rootProject); - } - } + // Validate root projects that is not being validated yet. + rootProjects.removeAll(validatingProjects); if(rootProjects.isEmpty()) { // We don't have projects to validate. return OK_STATUS; } + validatingProjects.addAll(rootProjects); } try { validationContextManager.clearValidatedProjectsList(); @@ -96,9 +91,7 @@ validationHelper.cleanup(); // See https://issues.jboss.org/browse/JBIDE-8726 } finally { synchronized (validatingProjects) { - for (IProject rootProject : rootProjects) { - validatingProjects.remove(rootProject); - } + validatingProjects.removeAll(rootProjects); } } }