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-lang.git
The following commit(s) were added to refs/heads/master by this push: new 3087acb92 Simplifiable conditional expression. (#941) 3087acb92 is described below commit 3087acb92951fa4e4805761dfe34c0245588303b Author: Arturo Bernal <arturobern...@gmail.com> AuthorDate: Sun Aug 21 14:51:09 2022 +0200 Simplifiable conditional expression. (#941) --- src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java | 2 +- src/main/java/org/apache/commons/lang3/stream/Streams.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java b/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java index eb62b56da..769cb2f8d 100644 --- a/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java +++ b/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java @@ -101,7 +101,7 @@ final class MemberUtils { * @return {@code true} if {@code m} is accessible */ static boolean isAccessible(final Member member) { - return member != null && isPublic(member) && !member.isSynthetic(); + return isPublic(member) && !member.isSynthetic(); } /** diff --git a/src/main/java/org/apache/commons/lang3/stream/Streams.java b/src/main/java/org/apache/commons/lang3/stream/Streams.java index 29a00a726..c8455003d 100644 --- a/src/main/java/org/apache/commons/lang3/stream/Streams.java +++ b/src/main/java/org/apache/commons/lang3/stream/Streams.java @@ -165,7 +165,7 @@ public class Streams { @Override public boolean tryAdvance(final Consumer<? super T> action) { - return enumeration.hasMoreElements() ? next(action) : false; + return enumeration.hasMoreElements() && next(action); } }