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 2013/06/24 11:01:48 UTC

svn commit: r1495963 - in /jena/trunk/jena-arq/src: main/java/com/hp/hpl/jena/sparql/algebra/OpVars.java test/java/com/hp/hpl/jena/sparql/algebra/optimize/TestTransformFilters.java

Author: andy
Date: Mon Jun 24 09:01:47 2013
New Revision: 1495963

URL: http://svn.apache.org/r1495963
Log:
Remove warnings

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

Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpVars.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpVars.java?rev=1495963&r1=1495962&r2=1495963&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpVars.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpVars.java Mon Jun 24 09:01:47 2013
@@ -64,6 +64,7 @@ public class OpVars
         OpWalker.walk(new WalkerVisitorVisible(visitor, acc), op) ;
     }
     
+    @SuppressWarnings("unchecked")
     public static Tuple<Set<Var>> mentionedVarsByPosition(Op op) {
         Set<Var> graphAcc = collector() ;
         Set<Var> subjAcc = collector() ;
@@ -75,6 +76,7 @@ public class OpVars
         return Tuple.create(graphAcc, subjAcc, predAcc, objAcc, unknownAcc);
     }
     
+    @SuppressWarnings("unchecked")
     public static Tuple<Set<Var>> mentionedVarsByPosition(Op... ops) {
         Set<Var> graphAcc = collector() ;
         Set<Var> subjAcc = collector() ;
@@ -82,9 +84,8 @@ public class OpVars
         Set<Var> objAcc = collector() ;
         Set<Var> unknownAcc = collector() ;
         OpVarsPatternWithPositions visitor = new OpVarsPatternWithPositions(graphAcc, subjAcc, predAcc, objAcc, unknownAcc, false);
-        for (Op op : ops) {
+        for (Op op : ops)
             OpWalker.walk(op, visitor);
-        }
         return Tuple.create(graphAcc, subjAcc, predAcc, objAcc, unknownAcc);
     }
 
@@ -392,7 +393,6 @@ public class OpVars
         addVar(acc, t.getObject()) ;
     }
 
-    @SuppressWarnings("unused")
     private static void addVarsFromQuad(Collection<Var> acc, Quad q) {
         addVar(acc, q.getSubject()) ;
         addVar(acc, q.getPredicate()) ;

Modified: jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/optimize/TestTransformFilters.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/optimize/TestTransformFilters.java?rev=1495963&r1=1495962&r2=1495963&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/optimize/TestTransformFilters.java (original)
+++ jena/trunk/jena-arq/src/test/java/com/hp/hpl/jena/sparql/algebra/optimize/TestTransformFilters.java Mon Jun 24 09:01:47 2013
@@ -583,7 +583,7 @@ public class TestTransformFilters
         test(
              "(filter (= ?x ?x) (bgp (?x ?p ?o)(?y ?p1 ?o1)))",
              t_implicitJoin,
-             null);
+             (String[])null);
     }
         
     @Test public void implicitLeftJoin1()
@@ -712,7 +712,7 @@ public class TestTransformFilters
         test(
              "(leftjoin (bgp (?x ?p ?o)(?x <http://pred> ?y)) (bgp (?a ?b ?c)) ((= ?x ?y)))",
              t_implicitLeftJoin,
-             null);
+             (String[])null);
     }
     
     @Test public void implicitLeftJoin9()
@@ -730,7 +730,7 @@ public class TestTransformFilters
         test(
              "(leftjoin (bgp (?x ?p ?o)) (bgp (?y ?p1 ?o1)) (|| (= ?x ?y) (> ?o1 ?o2)))",
              t_implicitLeftJoin,
-             null);
+             (String[])null);
     }
     
     @Test public void implicitLeftJoinConditional1()