From 5acfd4e1beaf73f55397404c2f532395f1689fd8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Yann=20C=C3=A9bron?= Date: Wed, 12 Sep 2012 12:58:13 +0200 Subject: [PATCH] XmlLanguageInjector.getInjectedLanguage: a bit of performance for XmlAttribute --- .../intelliLang/inject/xml/XmlLanguageInjector.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/plugins/IntelliLang/xml-support/org/intellij/plugins/intelliLang/inject/xml/XmlLanguageInjector.java b/plugins/IntelliLang/xml-support/org/intellij/plugins/intelliLang/inject/xml/XmlLanguageInjector.java index a0f692155dbb..e9bc5515ce93 100644 --- a/plugins/IntelliLang/xml-support/org/intellij/plugins/intelliLang/inject/xml/XmlLanguageInjector.java +++ b/plugins/IntelliLang/xml-support/org/intellij/plugins/intelliLang/inject/xml/XmlLanguageInjector.java @@ -164,7 +164,10 @@ public final class XmlLanguageInjector implements MultiHostInjector { return; } - for (BaseInjection injection : myConfiguration.getInjections(LanguageInjectionSupport.XML_SUPPORT_ID)) { + List injections = myConfiguration.getInjections(LanguageInjectionSupport.XML_SUPPORT_ID); + //noinspection ForLoopReplaceableByForEach + for (int i = 0, size = injections.size(); i < size; i++) { + BaseInjection injection = injections.get(i); if (injection.acceptsPsiElement(attribute)) { final Language language = InjectedLanguage.findLanguageById(injection.getInjectedLanguageId()); if (language == null) continue; @@ -173,7 +176,8 @@ public final class XmlLanguageInjector implements MultiHostInjector { final List ranges = injection.getInjectedArea(value); if (ranges.isEmpty()) continue; final List> result = new ArrayList>(); - final InjectedLanguage l = InjectedLanguage.create(injection.getInjectedLanguageId(), injection.getPrefix(), injection.getSuffix(), false); + final InjectedLanguage l = + InjectedLanguage.create(injection.getInjectedLanguageId(), injection.getPrefix(), injection.getSuffix(), false); for (TextRange textRange : ranges) { result.add(Trinity.create((PsiLanguageInjectionHost)value, l, textRange)); }