You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/08/21 12:51:17 UTC
[commons-lang] branch master updated: Simplifiable conditional expression. (#941)
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 <ar...@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);
}
}