Author: vrubezhny
Date: 2012-07-09 12:57:20 -0400 (Mon, 09 Jul 2012)
New Revision: 42469
Modified:
trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java/JavaDirtyRegionProcessor.java
Log:
JBIDE-10611
As-you-type CDI validation
CDI-like validators are invoked for an original region instead of calculated start and end
partitions (which was a much wider region)
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-07-09
16:33:01 UTC (rev 42468)
+++
trunk/common/plugins/org.jboss.tools.common.validation/src/org/jboss/tools/common/validation/java/JavaDirtyRegionProcessor.java 2012-07-09
16:57:20 UTC (rev 42469)
@@ -148,7 +148,7 @@
return fCompilationUnit;
}
- private void clearAllAnnotations() {
+ public void clearAllAnnotations() {
if (fAnnotations.isEmpty()) {
return;
}
@@ -310,6 +310,7 @@
public void uninstall() {
fIsCanceled = true;
if(fReporter != null) {
+ fReporter.clearAllAnnotations();
fReporter.setCanceled(true);
}
@@ -393,11 +394,21 @@
fReporter.clearAnnotations(fStartPartitionsToProcess, fEndPartitionsToProcess);
for (ITypedRegion partition : fPartitionsToProcess) {
+// try {
+// System.out.println("validateString: " + partition.getOffset() +
"->" + (partition.getOffset() + partition.getLength()) + ": [" +
fDocument.get(partition.getOffset(), partition.getLength())+ "]");
+// } catch (BadLocationException e) {
+// e.printStackTrace();
+// }
fValidatorManager.validateString(partition, fHelper, fReporter);
}
if (isJavaElementValidationRequired(fStartRegionToProcess, fEndRegionToProcess)) {
- fValidatorManager.validateJavaElement(new Region(fStartPartitionsToProcess,
fEndPartitionsToProcess - fStartPartitionsToProcess), fHelper, fReporter);
+// try {
+// System.out.println("validateJavaElement: " + fStartRegionToProcess +
"->" + fEndRegionToProcess + ": [" +
fDocument.get(fStartRegionToProcess, fEndRegionToProcess - fStartRegionToProcess)+
"]");
+// } catch (BadLocationException e) {
+// e.printStackTrace();
+// }
+ fValidatorManager.validateJavaElement(new Region(fStartRegionToProcess,
fEndRegionToProcess - fStartRegionToProcess), fHelper, fReporter);
}
}
Show replies by date