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/03/21 21:12:19 UTC

svn commit: r1459505 - /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java

Author: andy
Date: Thu Mar 21 20:12:18 2013
New Revision: 1459505

URL: http://svn.apache.org/r1459505
Log:
Fix comments

Modified:
    jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java

Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java?rev=1459505&r1=1459504&r2=1459505&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java Thu Mar 21 20:12:18 2013
@@ -531,7 +531,7 @@ public class Query extends Prologue impl
     // Record allocated aggregations.
     // Later: The same aggregation expression used in a query 
     // will always lead to the same aggregator.
-    // For now, allocate a fresh one each time (cause the calcutation
+    // For now, allocate a fresh one each time (cause the calculation
     // to be done multiple times but (1) it's unusual to have repeated 
     // aggregators normally and (2) the actual calculation is cheap. 
         
@@ -539,7 +539,6 @@ public class Query extends Prologue impl
     // Commonality?
     
     private List<ExprAggregator> aggregators = new ArrayList<ExprAggregator>() ;
-    // Using a LinkedhashMap does seem to give strong enugh hashCode equality. 
     private Map<Var, ExprAggregator> aggregatorsMap = new HashMap<Var, ExprAggregator>() ;
     
     // Note any E_Aggregator created for reuse.
@@ -551,7 +550,7 @@ public class Query extends Prologue impl
     public Expr allocAggregate(Aggregator agg)
     {
         // We need to track the aggregators in case one aggregator is used twice, e.g. in HAVING and in SELECT expression
-        // (is is that much harm to do twice?  Yes, if distinct.)
+        // (is that much harm to do twice?  Yes, if distinct.)
         String key = agg.key() ;
         
         Var v = aggregatorsAllocated.get(key);