You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bval.apache.org by mb...@apache.org on 2018/03/30 17:45:02 UTC
[4/5] bval git commit: avoid redundant path copy operations
avoid redundant path copy operations
Project: http://git-wip-us.apache.org/repos/asf/bval/repo
Commit: http://git-wip-us.apache.org/repos/asf/bval/commit/a32278a3
Tree: http://git-wip-us.apache.org/repos/asf/bval/tree/a32278a3
Diff: http://git-wip-us.apache.org/repos/asf/bval/diff/a32278a3
Branch: refs/heads/bv2
Commit: a32278a30191eaf007b3a9d215c2efef05fc40e9
Parents: 78c76ba
Author: Matt Benson <mb...@apache.org>
Authored: Fri Mar 30 12:42:14 2018 -0500
Committer: Matt Benson <mb...@apache.org>
Committed: Fri Mar 30 12:42:14 2018 -0500
----------------------------------------------------------------------
.../src/main/java/org/apache/bval/jsr/descriptor/PropertyD.java | 2 +-
.../src/main/java/org/apache/bval/jsr/job/ValidationJob.java | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/bval/blob/a32278a3/bval-jsr/src/main/java/org/apache/bval/jsr/descriptor/PropertyD.java
----------------------------------------------------------------------
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 4b1d5f9..38cfaaa 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
@@ -100,7 +100,7 @@ public abstract class PropertyD<E extends AnnotatedElement> extends CascadableCo
}
try {
final Object value = getValue(context.getValue());
- final PathImpl p = PathImpl.copy(context.getPath());
+ final PathImpl p = context.getPath();
p.addProperty(getPropertyName());
return Stream.of(context.child(p, value));
} catch (Exception e) {
http://git-wip-us.apache.org/repos/asf/bval/blob/a32278a3/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java
----------------------------------------------------------------------
diff --git a/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java b/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java
index 7cb3538..6542717 100644
--- a/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java
+++ b/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java
@@ -283,7 +283,7 @@ public abstract class ValidationJob<T> {
final TraversableResolver traversableResolver = validatorContext.getTraversableResolver();
final Stream<PropertyD<?>> reachableProperties = properties.filter(d -> {
- final PathImpl p = PathImpl.copy(context.getPath());
+ final PathImpl p = context.getPath();
p.addProperty(d.getPropertyName());
try {
return traversableResolver.isReachable(context.getValue(), p.removeLeafNode(), getRootBeanClass(),
@@ -337,7 +337,7 @@ public abstract class ValidationJob<T> {
final Object traversableObject =
Optional.ofNullable(context.getParent()).map(GraphContext::getValue).orElse(null);
- final PathImpl pathToTraversableObject = PathImpl.copy(context.getPath());
+ final PathImpl pathToTraversableObject = context.getPath();
final NodeImpl traversableProperty = pathToTraversableObject.removeLeafNode();
try {