This is an automated email from the ASF dual-hosted git repository. olli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-scripting-freemarker.git
commit 5cb0046b3353431e5b25be7d2c513657596f9f18 Author: Oliver Lietz <[email protected]> AuthorDate: Fri Nov 4 12:28:38 2022 +0100 style --- .../freemarker/internal/FreemarkerScriptEngineFactory.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/apache/sling/scripting/freemarker/internal/FreemarkerScriptEngineFactory.java b/src/main/java/org/apache/sling/scripting/freemarker/internal/FreemarkerScriptEngineFactory.java index 7776b79..a6403c7 100644 --- a/src/main/java/org/apache/sling/scripting/freemarker/internal/FreemarkerScriptEngineFactory.java +++ b/src/main/java/org/apache/sling/scripting/freemarker/internal/FreemarkerScriptEngineFactory.java @@ -146,10 +146,8 @@ public final class FreemarkerScriptEngineFactory extends AbstractScriptEngineFac final String namespace = (String) serviceReference.getProperty("namespace"); final String name = (String) serviceReference.getProperty("name"); if (StringUtils.isNotBlank(namespace) && StringUtils.isNotBlank(name)) { - if (!models.containsKey(namespace)) { - models.put(namespace, new HashMap<>()); - } - models.get(namespace).put(name, bundleContext.getService(serviceReference)); + final Map<String, TemplateModel> map = models.computeIfAbsent(namespace, k -> new HashMap<>()); + map.put(name, bundleContext.getService(serviceReference)); } } return models;
