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());
     }
 

Reply via email to