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 79290904a0c9491ace93992463b83ed073e55706 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sat Jul 16 12:15:15 2022 -0400 Use forEach() --- .../configuration2/builder/BasicBuilderParameters.java | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/main/java/org/apache/commons/configuration2/builder/BasicBuilderParameters.java b/src/main/java/org/apache/commons/configuration2/builder/BasicBuilderParameters.java index c8482255..8b6833fe 100644 --- a/src/main/java/org/apache/commons/configuration2/builder/BasicBuilderParameters.java +++ b/src/main/java/org/apache/commons/configuration2/builder/BasicBuilderParameters.java @@ -242,11 +242,11 @@ public class BasicBuilderParameters implements Cloneable, BuilderParameters, Bas throw new IllegalArgumentException("Parameters to merge must not be null!"); } - for (final Map.Entry<String, Object> e : p.getParameters().entrySet()) { + p.getParameters().entrySet().forEach(e -> { if (!properties.containsKey(e.getKey()) && !e.getKey().startsWith(RESERVED_PARAMETER_PREFIX)) { storeProperty(e.getKey(), e.getValue()); } - } + }); } /** @@ -423,11 +423,11 @@ public class BasicBuilderParameters implements Cloneable, BuilderParameters, Bas return null; } - for (final Map.Entry<?, ?> e : prefixes.entrySet()) { + prefixes.entrySet().forEach(e -> { if (!(e.getKey() instanceof String) || !(e.getValue() instanceof Lookup)) { throw new IllegalArgumentException("Map with prefix lookups contains invalid data: " + prefixes); } - } + }); return fetchPrefixLookups(params); } @@ -460,10 +460,8 @@ public class BasicBuilderParameters implements Cloneable, BuilderParameters, Bas return null; } - for (final Object o : col) { - if (!(o instanceof Lookup)) { - throw new IllegalArgumentException("Collection with default lookups contains invalid data: " + col); - } + if (col.stream().noneMatch(o -> o instanceof Lookup)) { + throw new IllegalArgumentException("Collection with default lookups contains invalid data: " + col); } return fetchDefaultLookups(params); }