You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by rv...@apache.org on 2013/03/25 22:58:11 UTC
svn commit: r1460912 -
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpAsQuery.java
Author: rvesse
Date: Mon Mar 25 21:58:11 2013
New Revision: 1460912
URL: http://svn.apache.org/r1460912
Log:
Remove check rendered uncesseary by change to using a Set instead of a List which also happens to be the source of the irrelevant warnings (JENA-422)
Modified:
jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpAsQuery.java
Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpAsQuery.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpAsQuery.java?rev=1460912&r1=1460911&r2=1460912&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpAsQuery.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/algebra/OpAsQuery.java Mon Mar 25 21:58:11 2013
@@ -55,12 +55,7 @@ public class OpAsQuery
query.setQueryResultStar(vars.isEmpty()); // SELECT * unless we are projecting
Iterator<Var> iter = vars.iterator();
for (; iter.hasNext();) {
- Var var = iter.next();
-
- // Depending on where the variable comes from we may already
- // have added this as a result variable
- if (query.getResultVars().contains(var)) continue;
-
+ Var var = iter.next();
if (v.varExpression.containsKey(var))
query.addResultVar(var, v.varExpression.get(var));
else