This is an automated email from the ASF dual-hosted git repository. tv pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/turbine-core.git
commit faa1d8ab15e28544b4ddad5da02fb930ac8403b8 Author: Thomas Vandahl <[email protected]> AuthorDate: Sun Dec 14 20:48:03 2025 +0100 Use ConcurrentMap --- .../assemblerbroker/TurbineAssemblerBrokerService.java | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/src/java/org/apache/turbine/services/assemblerbroker/TurbineAssemblerBrokerService.java b/src/java/org/apache/turbine/services/assemblerbroker/TurbineAssemblerBrokerService.java index 94c2ae99..4ac606da 100644 --- a/src/java/org/apache/turbine/services/assemblerbroker/TurbineAssemblerBrokerService.java +++ b/src/java/org/apache/turbine/services/assemblerbroker/TurbineAssemblerBrokerService.java @@ -62,7 +62,7 @@ public class TurbineAssemblerBrokerService = LogManager.getLogger(TurbineAssemblerBrokerService.class); /** A structure that holds the registered AssemblerFactories */ - private Map<Class<?>, List<?>> factories = null; + private ConcurrentMap<Class<?>, List<?>> factories = null; /** A cache that holds the generated Assemblers */ private ConcurrentMap<String, Assembler> assemblerCache = null; @@ -85,11 +85,8 @@ public class TurbineAssemblerBrokerService @SuppressWarnings("unchecked") private <T extends Assembler> List<AssemblerFactory<T>> getFactoryGroup(Class<T> type) { - if (!factories.containsKey(type)) - { - factories.put(type, new ArrayList<AssemblerFactory<T>>()); - } - return (List<AssemblerFactory<T>>) factories.get(type); + return (List<AssemblerFactory<T>>) factories.computeIfAbsent(type, + k -> new ArrayList<AssemblerFactory<T>>()); } /** @@ -140,7 +137,7 @@ public class TurbineAssemblerBrokerService public void init() throws InitializationException { - factories = new HashMap<>(); + factories = new ConcurrentHashMap<>(); try {
