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 2014/04/28 12:33:22 UTC

svn commit: r1590591 - /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformFilterPlacementConservative.java

Author: andy
Date: Mon Apr 28 10:33:21 2014
New Revision: 1590591

URL: http://svn.apache.org/r1590591
Log:
JENA-681 -- Remove commented out code.

Modified:
    jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformFilterPlacementConservative.java

Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformFilterPlacementConservative.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformFilterPlacementConservative.java?rev=1590591&r1=1590590&r2=1590591&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformFilterPlacementConservative.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/optimize/TransformFilterPlacementConservative.java Mon Apr 28 10:33:21 2014
@@ -70,29 +70,6 @@ public class TransformFilterPlacementCon
     
     public TransformFilterPlacementConservative( ) {}
 
-    // Original helper statics.
-    // Removed because this is not the best TransformFilterPlacement to use.
-//    public static Op transform(ExprList exprs, BasicPattern bgp) {
-//        if (!doFilterPlacement)
-//            return OpFilter.filter(exprs, new OpBGP(bgp));
-//        // Mutated
-//        ExprList exprs2 = ExprList.copy(exprs);
-//        Op op = transformFilterBGP(exprs2, new HashSet<Var>(), bgp);
-//        // Remaining filters? e.g. ones mentioning var s not used anywhere.
-//        op = buildFilter(exprs2, op);
-//        return op;
-//    }
-//
-//    public static Op transform(ExprList exprs, Node graphNode, BasicPattern bgp) {
-//        if (!doFilterPlacement)
-//            return OpFilter.filter(exprs, new OpQuadPattern(graphNode, bgp));
-//        // Mutated
-//        ExprList exprs2 = ExprList.copy(exprs);
-//        Op op = transformFilterQuadPattern(exprs2, new HashSet<Var>(), graphNode, bgp);
-//        op = buildFilter(exprs2, op);
-//        return op;
-//    }
-
     @Override
     public Op transform(OpFilter opFilter, Op x) {
         // Destructive use of exprs - copy it.