Use cache by default (see #159).

eclipse-lemminx · Oct 29, 2018 · 8a5444e · 8a5444e
1 parent 6055aef
commit 8a5444e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/.../src/main/java/org/eclipse/lsp4xml/extensions/contentmodel/model/ContentModelManager.java b/.../src/main/java/org/eclipse/lsp4xml/extensions/contentmodel/model/ContentModelManager.java
@@ -55,12 +55,12 @@ public static ContentModelManager getInstance() {
 	private final XMLFileAssociationResolverExtension fileAssociationResolver;
 
 	public ContentModelManager() {
-		cmDocumentCache = new HashMap<>();		
+		cmDocumentCache = new HashMap<>();
 		URIResolverExtensionManager resolverManager = URIResolverExtensionManager.getInstance();
 		loader = new XSLoaderImpl();
 		loader.setParameter("http://apache.org/xml/properties/internal/entity-resolver", resolverManager);
 		loader.setParameter(Constants.DOM_ERROR_HANDLER, new DOMErrorHandler() {
-			
+
 			@Override
 			public boolean handleError(DOMError error) {
 				if (error.getRelatedException() instanceof CacheResourceDownloadingException) {
@@ -75,6 +75,8 @@ public boolean handleError(DOMError error) {
 		resolverManager.registerResolver(fileAssociationResolver);
 		catalogResolverExtension = new XMLCatalogResolverExtension();
 		resolverManager.registerResolver(catalogResolverExtension);
+		// Use cache by default
+		setUseCache(true);
 	}
 
 	public CMElementDeclaration findCMElement(Element element) throws Exception {

diff --git a/.../main/java/org/eclipse/lsp4xml/extensions/contentmodel/settings/ContentModelSettings.java b/.../main/java/org/eclipse/lsp4xml/extensions/contentmodel/settings/ContentModelSettings.java
@@ -24,11 +24,6 @@ public class ContentModelSettings {
 
 	private XMLFileAssociation[] fileAssociations;
 
-	public ContentModelSettings() {
-		// Use cache by default
-		setUseCache(true);
-	}
-
 	/**
 	 * Returns true if cache to download XML Schema, DTD must be activated and false
 	 * otherwise.