Minor bug fixes.

gkellogg · Feb 15, 2011 · 4c6fb5f · 4c6fb5f
1 parent afdd054
commit 4c6fb5f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/lib/sparql/algebra/operator/left_join.rb b/lib/sparql/algebra/operator/left_join.rb
@@ -4,7 +4,7 @@ class Operator
     # The SPARQL GraphPattern `leftJoin` operator.
     #
     # @see http://www.w3.org/TR/rdf-sparql-query/#sparqlAlgebra
-   class LeftJoin < Operator
+    class LeftJoin < Operator
       include Query
 
       NAME = [:leftjoin]
@@ -50,7 +50,7 @@ def execute(queryable, options = {})
       # @return [Union, RDF::Query] `self`
       def optimize
         ops = operands.map {|o| o.optimize }.select {|o| o.respond_to?(:empty?) && !o.empty?}
-        expr = ops.pop if unless ops.last.executable?
+        expr = ops.pop unless ops.last.executable?
         expr = nil if expr.respond_to?(:true?) && expr.true?
 
         # ops now is one or two executable operators

diff --git a/lib/sparql/algebra/operator/prefix.rb b/lib/sparql/algebra/operator/prefix.rb
@@ -4,7 +4,7 @@ class Operator
     # The SPARQL GraphPattern `prefix` operator.
     #
     # @see http://www.w3.org/TR/xpath-functions/#func-numeric-add
-    class Prefix < Operator::Binary
+    class Prefix < Binary
       include Query
 
       NAME = [:prefix]
@@ -22,7 +22,7 @@ class Prefix < Operator::Binary
       #   the resulting solution sequence
       # @see    http://www.w3.org/TR/rdf-sparql-query/#sparqlAlgebra
       def execute(queryable, options = {})
-        @solutions = operands.last.execute(queriable, options = {})
+        @solutions = operands.last.execute(queryable, options = {})
       end
 
       ##

diff --git a/lib/sparql/algebra/operator/project.rb b/lib/sparql/algebra/operator/project.rb
@@ -23,8 +23,8 @@ class Project < Operator::Binary
       # @see    http://www.w3.org/TR/rdf-sparql-query/#sparqlAlgebra
       def execute(queryable, options = {})
         @solutions = operands.last.
-          execute(queriable, options = {}).
-          project(operands.first)
+          execute(queryable, options = {}).
+          project(*(operands.first))
       end
 
       ##