fetch supported locales only once instead of three times (#2276)

Signed-off-by: Markus Rathgeb <maggu2810@gmail.com>
eclipse-archived · Oct 7, 2016 · 6e3d779 · 6e3d779
1 parent 354ed5c
commit 6e3d779
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/...e.io.rest.voice/src/main/java/org/eclipse/smarthome/io/rest/voice/internal/HLIMapper.java b/...e.io.rest.voice/src/main/java/org/eclipse/smarthome/io/rest/voice/internal/HLIMapper.java
@@ -9,6 +9,7 @@
 
 import java.util.HashSet;
 import java.util.Locale;
+import java.util.Set;
 
 import org.eclipse.smarthome.core.voice.text.HumanLanguageInterpreter;
 
@@ -32,10 +33,11 @@ public static HumanLanguageInterpreterDTO map(HumanLanguageInterpreter hli, Loca
  HumanLanguageInterpreterDTO dto = new HumanLanguageInterpreterDTO();
  dto.id = hli.getId();
  dto.label = hli.getLabel(locale);
- if (hli.getSupportedLocales() != null) {
- dto.locales = new HashSet<String>(hli.getSupportedLocales().size());
- for (Locale l : hli.getSupportedLocales()) {
- dto.locales.add(l.toString());
+ final Set<Locale> supportedLocales = hli.getSupportedLocales();
+ if (supportedLocales != null) {
+ dto.locales = new HashSet<String>(supportedLocales.size());
+ for (final Locale supportedLocale : supportedLocales) {
+ dto.locales.add(supportedLocale.toString());
  }
  }
  return dto;