100% CPU Usage since version 0.18.3

Fixes eclipse-lemminx#1175

Signed-off-by: azerr <[email protected]>

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/XMLLanguageService.java

@@ -168,6 +168,11 @@ public List<Diagnostic> doDiagnostics(DOMDocument xmlDocument, XMLValidationSett
 	public CompletableFuture<Path> publishDiagnostics(DOMDocument xmlDocument,
 			Consumer<PublishDiagnosticsParams> publishDiagnostics, Consumer<TextDocument> triggerValidation,
 			XMLValidationSettings validationSettings, CancelChecker cancelChecker) {
+		if (validationSettings != null && !validationSettings.isEnabled()) {
+			// Validation is disabled
+			return null;
+		}
+		// Process validation
 		String uri = xmlDocument.getDocumentURI();
 		TextDocument document = xmlDocument.getTextDocument();
 
@@ -271,7 +276,7 @@ public Position getMatchingTagPosition(DOMDocument xmlDocument, Position positio
 	/**
 	 * Returns the linked editing ranges for the given <code>xmlDocument</code> at
 	 * the given <code>position</code> and null otherwise.
-	 * 
+	 *
 	 * @param xmlDocument   the DOM document.
 	 * @param position      the position.
 	 * @param cancelChecker the cancel checker.

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/extensions/diagnostics/DiagnosticsResult.java

@@ -21,15 +21,15 @@
 
 /**
  * This class is the result of a diagnostic process. It contains:
- * 
+ *
  * <ul>
  * <li>list of diagnostics.</li>
  * <li>list of completable future which are not done(ex : download some external
  * resources XSD, DTD). This list of future gives the capability to refresh
  * again the diagnostics once all completable futures are finished (ex : all
  * download are finished).</li>
  * </ul>
- * 
+ *
  * @author Angelo ZERR
  *
  */
@@ -41,7 +41,7 @@ public class DiagnosticsResult extends ArrayList<Diagnostic> {
 
 	static {
 		EMPTY = new DiagnosticsResult();
-		EMPTY.futures = Collections.emptyList();
+		EMPTY.futures = null;
 	}
 
 	private transient List<CompletableFuture<?>> futures;
@@ -56,7 +56,7 @@ public void addFuture(CompletableFuture<?> future) {
 	/**
 	 * Returns the completable futures used in a diagnostics (ex : completeable
 	 * future to download external resources XSD, DTD) and null otherwise.
-	 * 
+	 *
 	 * @return the completable futures used in a diagnostics (ex : completeable
 	 *         future to download external resources XSD, DTD) and null otherwise.
 	 */