This is an automated email from the ASF dual-hosted git repository.
rmannibucau pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bval.git
The following commit(s) were added to refs/heads/master by this push:
new 43935b4 drop validations which should be redundant due to code path
43935b4 is described below
commit 43935b44a95164fda045357ac969288696af6ba6
Author: Romain Manni-Bucau <[email protected]>
AuthorDate: Sun Feb 17 12:20:26 2019 +0100
drop validations which should be redundant due to code path
---
bval-jsr/src/main/java/org/apache/bval/jsr/GraphContext.java | 12 +++++-------
.../main/java/org/apache/bval/jsr/descriptor/PropertyD.java | 2 --
bval-jsr/src/main/java/org/apache/bval/jsr/groups/Group.java | 3 +--
.../main/java/org/apache/bval/jsr/groups/GroupStrategy.java | 6 ------
.../org/apache/bval/jsr/valueextraction/ValueExtractors.java | 1 -
5 files changed, 6 insertions(+), 18 deletions(-)
diff --git a/bval-jsr/src/main/java/org/apache/bval/jsr/GraphContext.java
b/bval-jsr/src/main/java/org/apache/bval/jsr/GraphContext.java
index 9ab4141..24ec601 100644
--- a/bval-jsr/src/main/java/org/apache/bval/jsr/GraphContext.java
+++ b/bval-jsr/src/main/java/org/apache/bval/jsr/GraphContext.java
@@ -30,7 +30,6 @@ import org.apache.bval.jsr.metadata.ContainerElementKey;
import org.apache.bval.jsr.util.NodeImpl;
import org.apache.bval.jsr.util.PathImpl;
import org.apache.bval.util.Exceptions;
-import org.apache.bval.util.Validate;
import org.apache.bval.util.reflection.TypeUtils;
public class GraphContext {
@@ -46,8 +45,8 @@ public class GraphContext {
private GraphContext(ApacheFactoryContext validatorContext, PathImpl path,
Object value, GraphContext parent) {
super();
- this.validatorContext = Validate.notNull(validatorContext,
"validatorContext");
- this.path = Validate.notNull(path, "path");
+ this.validatorContext = validatorContext;
+ this.path = path;
this.value = value;
this.parent = parent;
}
@@ -65,16 +64,16 @@ public class GraphContext {
}
public GraphContext child(NodeImpl node, Object value) {
- Validate.notNull(node, "node");
+ // Validate.notNull(node, "node");
final PathImpl p = PathImpl.copy(path);
p.addNode(node);
return new GraphContext(validatorContext, p, value, this);
}
public GraphContext child(Path p, Object value) {
- Validate.notNull(p, "Path");
+ // Validate.notNull(p, "Path");
final PathImpl impl = PathImpl.of(p);
- Validate.isTrue(impl.isSubPathOf(path), "%s is not a subpath of %s",
p, path);
+ // Validate.isTrue(impl.isSubPathOf(path), "%s is not a subpath of
%s", p, path);
return new GraphContext(validatorContext, impl == p ?
PathImpl.copy(impl) : impl, value, this);
}
@@ -120,7 +119,6 @@ public class GraphContext {
}
public ContainerElementKey runtimeKey(ContainerElementKey key) {
- Validate.notNull(key);
if (value != null) {
final Class<?> containerClass = key.getContainerClass();
final Class<? extends Object> runtimeType = value.getClass();
diff --git
a/bval-jsr/src/main/java/org/apache/bval/jsr/descriptor/PropertyD.java
b/bval-jsr/src/main/java/org/apache/bval/jsr/descriptor/PropertyD.java
index 38cfaaa..0b1a285 100644
--- a/bval-jsr/src/main/java/org/apache/bval/jsr/descriptor/PropertyD.java
+++ b/bval-jsr/src/main/java/org/apache/bval/jsr/descriptor/PropertyD.java
@@ -28,7 +28,6 @@ import javax.validation.metadata.PropertyDescriptor;
import org.apache.bval.jsr.GraphContext;
import org.apache.bval.jsr.util.Methods;
import org.apache.bval.jsr.util.PathImpl;
-import org.apache.bval.util.Validate;
import org.apache.bval.util.reflection.Reflection;
import org.apache.commons.weaver.privilizer.Privilizing;
import org.apache.commons.weaver.privilizer.Privilizing.CallTo;
@@ -94,7 +93,6 @@ public abstract class PropertyD<E extends AnnotatedElement>
extends CascadableCo
}
public final Stream<GraphContext> read(GraphContext context) {
- Validate.notNull(context);
if (context.getValue() == null) {
return Stream.empty();
}
diff --git a/bval-jsr/src/main/java/org/apache/bval/jsr/groups/Group.java
b/bval-jsr/src/main/java/org/apache/bval/jsr/groups/Group.java
index d7dd994..9092eb8 100644
--- a/bval-jsr/src/main/java/org/apache/bval/jsr/groups/Group.java
+++ b/bval-jsr/src/main/java/org/apache/bval/jsr/groups/Group.java
@@ -34,7 +34,6 @@ import javax.validation.groups.Default;
import org.apache.bval.util.Exceptions;
import org.apache.bval.util.ObjectWrapper;
-import org.apache.bval.util.Validate;
/**
* Immutable object that wraps an interface representing a single group.
@@ -128,7 +127,7 @@ public final class Group implements GroupStrategy {
* @param group
*/
public Group(Class<?> group) {
- this.group = Validate.notNull(group);
+ this.group = group;
}
/**
diff --git
a/bval-jsr/src/main/java/org/apache/bval/jsr/groups/GroupStrategy.java
b/bval-jsr/src/main/java/org/apache/bval/jsr/groups/GroupStrategy.java
index 0b53a26..343283c 100644
--- a/bval-jsr/src/main/java/org/apache/bval/jsr/groups/GroupStrategy.java
+++ b/bval-jsr/src/main/java/org/apache/bval/jsr/groups/GroupStrategy.java
@@ -29,7 +29,6 @@ import java.util.function.Predicate;
import java.util.stream.Collectors;
import org.apache.bval.jsr.util.ToUnmodifiable;
-import org.apache.bval.util.Validate;
/**
* Group strategy interface.
@@ -78,8 +77,6 @@ public interface GroupStrategy {
protected final boolean ordered;
public Composite(Collection<? extends GroupStrategy> components,
boolean ordered) {
-
Validate.isTrue(Validate.notNull(components).stream().noneMatch(Objects::isNull),
- "null component not permitted");
this.components = new LinkedHashSet<>(components);
this.ordered = ordered;
}
@@ -155,8 +152,6 @@ public interface GroupStrategy {
};
public static GroupStrategy redefining(GroupStrategy source, Map<Group, ?
extends GroupStrategy> redefinitions) {
- Validate.notNull(source, "source");
-
if (!(redefinitions == null || redefinitions.isEmpty())) {
if (redefinitions.containsValue(null)) {
redefinitions = redefinitions.entrySet().stream().filter(e ->
e.getValue() != null)
@@ -174,7 +169,6 @@ public interface GroupStrategy {
}
public static GroupStrategy simple(Collection<? extends Group> coll) {
- Validate.notNull(coll);
if (coll.size() == 1) {
return coll.iterator().next();
}
diff --git
a/bval-jsr/src/main/java/org/apache/bval/jsr/valueextraction/ValueExtractors.java
b/bval-jsr/src/main/java/org/apache/bval/jsr/valueextraction/ValueExtractors.java
index 430f267..89f50e3 100644
---
a/bval-jsr/src/main/java/org/apache/bval/jsr/valueextraction/ValueExtractors.java
+++
b/bval-jsr/src/main/java/org/apache/bval/jsr/valueextraction/ValueExtractors.java
@@ -257,7 +257,6 @@ public class ValueExtractors {
}
public void add(ValueExtractor<?> extractor) {
- Validate.notNull(extractor);
final ContainerElementKey key =
ContainerElementKey.forValueExtractor(extractor);
if (key == null) {
Exceptions.raise(IllegalStateException::new, "Computed null %s for
%s",
