You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cayenne.apache.org by km...@apache.org on 2008/01/30 03:12:48 UTC

svn commit: r616589 - /cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/access/DataDomainLegacyQueryAction.java

Author: kmenard
Date: Tue Jan 29 18:12:39 2008
New Revision: 616589

URL: http://svn.apache.org/viewvc?rev=616589&view=rev
Log:
Generics.

Modified:
    cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/access/DataDomainLegacyQueryAction.java

Modified: cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/access/DataDomainLegacyQueryAction.java
URL: http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/access/DataDomainLegacyQueryAction.java?rev=616589&r1=616588&r2=616589&view=diff
==============================================================================
--- cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/access/DataDomainLegacyQueryAction.java (original)
+++ cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/access/DataDomainLegacyQueryAction.java Tue Jan 29 18:12:39 2008
@@ -51,8 +51,8 @@
     Query query;
     QueryMetadata metadata;
 
-    Map queriesByNode;
-    Map queriesByExecutedQueries;
+    Map<QueryEngine, List<Query>> queriesByNode;
+    Map<Query, Query> queriesByExecutedQueries;
 
     DataDomainLegacyQueryAction(DataDomain domain, Query query, OperationObserver callback) {
         this.domain = domain;
@@ -75,11 +75,11 @@
 
         // run categorized queries
         if (queriesByNode != null) {
-            Iterator nodeIt = queriesByNode.entrySet().iterator();
+            Iterator<Map.Entry<QueryEngine,List<Query>>> nodeIt = queriesByNode.entrySet().iterator();
             while (nodeIt.hasNext()) {
-                Map.Entry entry = (Map.Entry) nodeIt.next();
-                QueryEngine nextNode = (QueryEngine) entry.getKey();
-                Collection nodeQueries = (Collection) entry.getValue();
+                Map.Entry<QueryEngine,List<Query>> entry = nodeIt.next();
+                QueryEngine nextNode = entry.getKey();
+                Collection<Query> nodeQueries = entry.getValue();
                 nextNode.performQueries(nodeQueries, this);
             }
         }
@@ -87,16 +87,16 @@
 
     public void route(QueryEngine engine, Query query, Query substitutedQuery) {
 
-        List queries = null;
+        List<Query> queries = null;
         if (queriesByNode == null) {
-            queriesByNode = new HashMap();
+            queriesByNode = new HashMap<QueryEngine, List<Query>>();
         }
         else {
-            queries = (List) queriesByNode.get(engine);
+            queries = queriesByNode.get(engine);
         }
 
         if (queries == null) {
-            queries = new ArrayList(5);
+            queries = new ArrayList<Query>(5);
             queriesByNode.put(engine, queries);
         }
 
@@ -107,7 +107,7 @@
         if (substitutedQuery != null && substitutedQuery != query) {
 
             if (queriesByExecutedQueries == null) {
-                queriesByExecutedQueries = new HashMap();
+                queriesByExecutedQueries = new HashMap<Query, Query>();
             }
 
             queriesByExecutedQueries.put(query, substitutedQuery);
@@ -164,7 +164,7 @@
         Query q = null;
 
         if (queriesByExecutedQueries != null) {
-            q = (Query) queriesByExecutedQueries.get(executedQuery);
+            q = queriesByExecutedQueries.get(executedQuery);
         }
 
         return q != null ? q : executedQuery;