You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2018/04/18 11:04:21 UTC

[2/5] jena git commit: Fix OpWalkerVisitor for OpExt

Fix OpWalkerVisitor for OpExt


Project: http://git-wip-us.apache.org/repos/asf/jena/repo
Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/9c2bb1c1
Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/9c2bb1c1
Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/9c2bb1c1

Branch: refs/heads/master
Commit: 9c2bb1c183a70fafa1007857823e91aca9a1f18f
Parents: 74529be
Author: Jeremy Coulon <je...@3ds.com>
Authored: Tue Apr 10 17:20:08 2018 +0200
Committer: Jeremy Coulon <je...@3ds.com>
Committed: Tue Apr 10 17:20:08 2018 +0200

----------------------------------------------------------------------
 .../src/main/java/org/apache/jena/sparql/algebra/OpVars.java    | 5 +++++
 .../src/main/java/org/apache/jena/sparql/algebra/OpWalker.java  | 1 +
 2 files changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/9c2bb1c1/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpVars.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpVars.java b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpVars.java
index 70f649e..9cfec1c 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpVars.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpVars.java
@@ -284,6 +284,11 @@ public class OpVars
         public void visit(OpProcedure opProc) {
             ExprVars.varsMentioned(acc, opProc.getArgs()) ;
         }
+        
+        @Override
+        public void visit(OpExt opExt) {
+            // OpWalkerVisitor is taking care of calling opExt.effectiveOp().visit(this)
+        }
     }
     
     private static class OpVarsPatternWithPositions extends OpVisitorBase

http://git-wip-us.apache.org/repos/asf/jena/blob/9c2bb1c1/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java
index 378e0ae..55bf4e7 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/algebra/OpWalker.java
@@ -139,6 +139,7 @@ public class OpWalker
         @Override
         protected void visitExt(OpExt op) {
             before(op) ;
+            super.visitExt(op);
             if ( visitor != null )
                 op.visit(visitor) ;
             after(op) ;