[jbosstools-commits] JBoss Tools SVN: r43418 - trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java.

[jbosstools-commits at lists.jboss.org]

Author: vrubezhny
Date: 2012-09-05 11:04:29 -0400 (Wed, 05 Sep 2012)
New Revision: 43418

Modified:
   trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java/JavaDirtyRegionProcessor.java
Log:
JBIDE-12539
In some cases as-you-type validation removes all annotations 

Issue is fixed

Modified: trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java/JavaDirtyRegionProcessor.java
===================================================================
--- trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java/JavaDirtyRegionProcessor.java	2012-09-05 14:21:37 UTC (rev 43417)
+++ trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java/JavaDirtyRegionProcessor.java	2012-09-05 15:04:29 UTC (rev 43418)
@@ -432,6 +432,7 @@
 			fValidatorManager.validateString(
 					Arrays.asList(fPartitionsToProcess.toArray(new IRegion[fPartitionsToProcess.size()])), 
 					fHelper, fReporter);
+			fReporter.finishReporting();
 		} else if (isJavaElementValidationRequired()) {
 			// The 'else' is added here due to not to validate 
 			// an element in case of at lease one string is validated,
@@ -443,9 +444,8 @@
 						new Region(fStartRegionToProcess, fEndRegionToProcess - fStartRegionToProcess)
 					}), 
 				fHelper, fReporter);
+			fReporter.finishReporting();
 		} 
-		
-		fReporter.finishReporting();
 	}
 
 	private boolean isJavaElementValidationRequired() {