Avoid useless computation of getMatchingRegexpAssociated that uses

Pattern (time consumming)

Author: RoiSoleil

runtime/bundles/org.eclipse.core.contenttype/src/org/eclipse/core/internal/content/ContentTypeCatalog.java

@@ -409,7 +409,7 @@ IContentType[] findContentTypesFor(ContentTypeMatcher matcher, InputStream conte
 	}
 
 	IContentType[] findContentTypesFor(ContentTypeMatcher matcher, final String fileName) {
-		IContentType[] selected = concat(internalFindContentTypesSorted(matcher, fileName, policyConstantGeneralIsBetter));
+		IContentType[] selected = concat(internalFindContentTypesSorted(matcher, fileName, policyConstantGeneralIsBetter ,false));
 		// give the policy a chance to change the results
 		ISelectionPolicy policy = matcher.getPolicy();
 		if (policy != null) {
@@ -418,6 +418,15 @@ IContentType[] findContentTypesFor(ContentTypeMatcher matcher, final String file
 		return selected;
 	}
 
+	IContentType findFirstContentTypeFor(ContentTypeMatcher matcher, final String fileName) {
+		ISelectionPolicy policy = matcher.getPolicy();
+		IContentType[] selected = concat(internalFindContentTypesSorted(matcher, fileName, policyConstantGeneralIsBetter, policy == null));
+		// give the policy a chance to change the results
+		if (policy != null)
+			selected = applyPolicy(policy, selected, true, false);
+		return selected.length != 0 ? selected[0] : null;
+	}
+
 	synchronized public IContentType[] getAllContentTypes() {
 		List<ContentType> result = new ArrayList<>(contentTypes.size());
 		for (IContentType iContentType : contentTypes.values()) {
@@ -555,7 +564,7 @@ private IContentType[] internalFindContentTypesFor(ContentTypeMatcher matcher, I
 			indeterminatePolicy = policyConstantGeneralIsBetter;
 			validPolicy = policyConstantSpecificIsBetter;
 		} else {
-			subset = internalFindContentTypesSorted(matcher, fileName, policyLexicographical);
+			subset = internalFindContentTypesSorted(matcher, fileName, policyLexicographical, false);
 			indeterminatePolicy = policyGeneralIsBetter;
 			validPolicy = policySpecificIsBetter;
 		}
@@ -590,7 +599,7 @@ private IContentType[] internalFindContentTypesFor(ContentTypeMatcher matcher, I
 	 * @return all matching content types in the preferred order
 	 * @see IContentTypeManager#findContentTypesFor(String)
 	 */
-	synchronized private IContentType[][] internalFindContentTypesSorted(ContentTypeMatcher matcher, final String fileName, Comparator<IContentType> sortingPolicy) {
+	synchronized private IContentType[][] internalFindContentTypesSorted(ContentTypeMatcher matcher, final String fileName, Comparator<IContentType> sortingPolicy, boolean quickFinish) {
 		IScopeContext context = matcher.getContext();
 		IContentType[][] result = { NO_CONTENT_TYPES, NO_CONTENT_TYPES, NO_CONTENT_TYPES };
 
@@ -609,6 +618,9 @@ synchronized private IContentType[][] internalFindContentTypesSorted(ContentType
 		result[0] = selectedByName.toArray(new IContentType[selectedByName.size()]);
 		if (result[0].length > 1) {
 			Arrays.sort(result[0], sortingPolicy);
+			if(quickFinish) {
+				return result;
+			}
 		}
 
 		final String fileExtension = ContentTypeManager.getFileExtension(fileName);
@@ -628,6 +640,9 @@ synchronized private IContentType[][] internalFindContentTypesSorted(ContentType
 		}
 		if (result[1].length > 1) {
 			Arrays.sort(result[1], sortingPolicy);
+			if (quickFinish) {
+				return result;
+			}
 		}
 
 		final Set<ContentType> allByFilePattern;

runtime/bundles/org.eclipse.core.contenttype/src/org/eclipse/core/internal/content/ContentTypeMatcher.java

@@ -47,8 +47,9 @@ public IContentType findContentTypeFor(InputStream contents, String fileName) th
 	public IContentType findContentTypeFor(String fileName) {
 		// basic implementation just gets all content types
 		ContentTypeCatalog currentCatalog = getCatalog();
-		IContentType[] associated = currentCatalog.findContentTypesFor(this, fileName);
-		return associated.length == 0 ? null : new ContentTypeHandler((ContentType) associated[0], currentCatalog.getGeneration());
+		IContentType associated = currentCatalog.findFirstContentTypeFor(this, fileName);
+		return associated == null ? null
+				: new ContentTypeHandler((ContentType) associated, currentCatalog.getGeneration());
 	}
 
 	@Override