This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-configuration.git
commit 57de54b518f8ad58d324467bca9dc20bd85ebb8c Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sat Jul 16 14:16:27 2022 -0400 Use forEach() --- .../DynamicCombinedConfiguration.java | 24 ++++++---------------- 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/src/main/java/org/apache/commons/configuration2/DynamicCombinedConfiguration.java b/src/main/java/org/apache/commons/configuration2/DynamicCombinedConfiguration.java index 85a53589..dba0e0f0 100644 --- a/src/main/java/org/apache/commons/configuration2/DynamicCombinedConfiguration.java +++ b/src/main/java/org/apache/commons/configuration2/DynamicCombinedConfiguration.java @@ -574,33 +574,25 @@ public class DynamicCombinedConfiguration extends CombinedConfiguration { @Override public void clearEventListeners() { - for (final CombinedConfiguration cc : configs.values()) { - cc.clearEventListeners(); - } + configs.values().forEach(CombinedConfiguration::clearEventListeners); super.clearEventListeners(); } @Override public <T extends Event> void addEventListener(final EventType<T> eventType, final EventListener<? super T> listener) { - for (final CombinedConfiguration cc : configs.values()) { - cc.addEventListener(eventType, listener); - } + configs.values().forEach(cc -> cc.addEventListener(eventType, listener)); super.addEventListener(eventType, listener); } @Override public <T extends Event> boolean removeEventListener(final EventType<T> eventType, final EventListener<? super T> listener) { - for (final CombinedConfiguration cc : configs.values()) { - cc.removeEventListener(eventType, listener); - } + configs.values().forEach(cc -> cc.removeEventListener(eventType, listener)); return super.removeEventListener(eventType, listener); } @Override public void clearErrorListeners() { - for (final CombinedConfiguration cc : configs.values()) { - cc.clearErrorListeners(); - } + configs.values().forEach(cc -> cc.clearErrorListeners()); super.clearErrorListeners(); } @@ -616,9 +608,7 @@ public class DynamicCombinedConfiguration extends CombinedConfiguration { } public void invalidateAll() { - for (final CombinedConfiguration cc : configs.values()) { - cc.invalidate(); - } + configs.values().forEach(CombinedConfiguration::invalidate); } /** @@ -733,9 +723,7 @@ public class DynamicCombinedConfiguration extends CombinedConfiguration { config.setConversionExpressionEngine(getConversionExpressionEngine()); config.setListDelimiterHandler(getListDelimiterHandler()); copyEventListeners(config); - for (final ConfigData data : configurations) { - config.addConfiguration(data.getConfiguration(), data.getName(), data.getAt()); - } + configurations.forEach(data -> config.addConfiguration(data.getConfiguration(), data.getName(), data.getAt())); config.setSynchronizer(getSynchronizer()); }