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 <[email protected]> Authored: Fri Mar 30 12:42:14 2018 -0500 Committer: Matt Benson <[email protected]> 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 {
