Index: dna-graph/pom.xml =================================================================== --- dna-graph/pom.xml (revision 1368) +++ dna-graph/pom.xml (working copy) @@ -32,11 +32,11 @@ joda-time joda-time - - com.google.code.google-collections - google-collect - snapshot-20080530 - + + com.google.collections + google-collections + 1.0-rc3 + Index: dna-graph/src/main/java/org/jboss/dna/graph/query/optimize/RewriteAsRangeCriteria.java =================================================================== --- dna-graph/src/main/java/org/jboss/dna/graph/query/optimize/RewriteAsRangeCriteria.java (revision 1368) +++ dna-graph/src/main/java/org/jboss/dna/graph/query/optimize/RewriteAsRangeCriteria.java (working copy) @@ -46,6 +46,7 @@ import org.jboss.dna.graph.query.plan.PlanNode; import org.jboss.dna.graph.query.plan.PlanNode.Property; import org.jboss.dna.graph.query.plan.PlanNode.Type; +import com.google.common.collect.ArrayListMultimap; import com.google.common.collect.Multimap; import com.google.common.collect.Multimaps; @@ -80,7 +81,7 @@ for (PlanNode access : plan.findAllAtOrBelow(Type.ACCESS)) { // Look for select nodes below an ACCESS node that have a single Comparison constraint, // and accumulate them keyed by the dynamic operand ... - Multimap selectNodeByOperand = Multimaps.newArrayListMultimap(); + Multimap selectNodeByOperand = ArrayListMultimap.create(); for (PlanNode select : access.findAllAtOrBelow(Type.SELECT)) { Constraint constraint = select.getProperty(Property.SELECT_CRITERIA, Constraint.class); // Look for Comparison constraints that use a range operator Index: dna-graph/src/main/java/org/jboss/dna/graph/session/GraphSession.java =================================================================== --- dna-graph/src/main/java/org/jboss/dna/graph/session/GraphSession.java (revision 1368) +++ dna-graph/src/main/java/org/jboss/dna/graph/session/GraphSession.java (working copy) @@ -66,6 +66,7 @@ import org.jboss.dna.graph.request.Request; import org.jboss.dna.graph.session.GraphSession.Authorizer.Action; import com.google.common.collect.ListMultimap; +import com.google.common.collect.LinkedListMultimap; import com.google.common.collect.Multimaps; /** @@ -86,7 +87,7 @@ @NotThreadSafe public class GraphSession { - protected final ListMultimap> NO_CHILDREN = Multimaps.immutableMultimap(); + protected final ListMultimap> NO_CHILDREN = LinkedListMultimap.create(); protected final Map> NO_PROPERTIES = Collections.emptyMap(); protected final Authorizer authorizer; @@ -1621,7 +1622,7 @@ if (children.isEmpty()) { childrenByName = cache.NO_CHILDREN; } else { - childrenByName = Multimaps.newLinkedListMultimap(); + childrenByName = LinkedListMultimap.create(); for (Location location : children) { NodeId id = cache.idFactory.create(); Name childName = location.getPath().getLastSegment().getName(); @@ -1699,7 +1700,7 @@ Name childName = childPath.getLastSegment().getName(); if (this.childrenByName.isEmpty()) { // Just have to add the child ... - this.childrenByName = Multimaps.newLinkedListMultimap(); + this.childrenByName = LinkedListMultimap.create(); if (childPath.getLastSegment().hasIndex()) { // The child has a SNS index, but this is an only child ... newChild = newChild.with(cache.pathFactory.create(childPath.getParent(), childName)); @@ -1710,7 +1711,7 @@ } // Unfortunately, there is no efficient way to insert into the multi-map, so we need to recreate it ... - ListMultimap> children = Multimaps.newLinkedListMultimap(); + ListMultimap> children = LinkedListMultimap.create(); boolean added = false; for (Node child : this.childrenByName.values()) { if (!added && child.isNew()) { @@ -1988,7 +1989,7 @@ child.remove(); // Now add the child ... if (parent.childrenByName == cache.NO_CHILDREN) { - parent.childrenByName = Multimaps.newLinkedListMultimap(); + parent.childrenByName = LinkedListMultimap.create(); } parent.childrenByName.put(newNodeName, child); child.parent = parent; @@ -2035,7 +2036,7 @@ parent.load(); if (parent.childrenByName == cache.NO_CHILDREN) { - parent.childrenByName = Multimaps.newLinkedListMultimap(); + parent.childrenByName = LinkedListMultimap.create(); } cache.nodeOperations.preCopy(this, parent); @@ -2101,7 +2102,7 @@ } // Unfortunately, there is no efficient way to insert into the multi-map, so we need to recreate it ... - ListMultimap> children = Multimaps.newLinkedListMultimap(); + ListMultimap> children = LinkedListMultimap.create(); for (Node child : childrenByName.values()) { if (child == nodeToBeMoved) continue; if (before != null && child.getSegment().equals(before)) { @@ -2388,7 +2389,7 @@ Node child = cache.createNode(this, cache.idFactory.create(), newChild); child.markAsNew(); // marks parent as changed if (childrenByName == cache.NO_CHILDREN) { - childrenByName = Multimaps.newLinkedListMultimap(); + childrenByName = LinkedListMultimap.create(); } childrenByName.put(name, child); Index: dna-graph/src/main/java/org/jboss/dna/graph/xml/XmlHandler.java =================================================================== --- dna-graph/src/main/java/org/jboss/dna/graph/xml/XmlHandler.java (revision 1368) +++ dna-graph/src/main/java/org/jboss/dna/graph/xml/XmlHandler.java (working copy) @@ -513,7 +513,7 @@ this.pathToParent = pathToParent; this.name = name; this.state = ElementEntryState.TBD; - properties = new LinkedHashMultimap(); + properties = LinkedHashMultimap.create(); } protected void setName( Name name ) { Index: dna-jcr/pom.xml =================================================================== --- dna-jcr/pom.xml (revision 1368) +++ dna-jcr/pom.xml (working copy) @@ -102,11 +102,11 @@ org.apache.jackrabbit jackrabbit-jcr-tests - - com.google.code.google-collections - google-collect - snapshot-20080530 - + + com.google.collections + google-collections + 1.0-rc3 + org.jboss.security Index: dna-jcr/src/main/java/org/jboss/dna/jcr/DefinitionCache.java =================================================================== --- dna-jcr/src/main/java/org/jboss/dna/jcr/DefinitionCache.java (revision 1368) +++ dna-jcr/src/main/java/org/jboss/dna/jcr/DefinitionCache.java (working copy) @@ -53,7 +53,7 @@ * first, followed by those of the immediate supertypes, followed by those from those supertypes' supertypes, etc. *

*/ - private final Multimap childNodeDefinitionsThatAllowSns = new LinkedListMultimap(); + private final Multimap childNodeDefinitionsThatAllowSns = LinkedListMultimap.create(); /** * A local cache of all defined and inherited child node definitions, keyed by their name, that do not allow * same-name-sibilings. This includes residual child node definitions, which are keyed by the @@ -67,7 +67,7 @@ * first, followed by those of the immediate supertypes, followed by those from those supertypes' supertypes, etc. *

*/ - private final Multimap childNodeDefinitionsThatAllowNoSns = new LinkedListMultimap(); + private final Multimap childNodeDefinitionsThatAllowNoSns = LinkedListMultimap.create(); /** * A local cache of all defined and inherited property definitions, keyed by their name, that allow multiple values. This @@ -80,7 +80,7 @@ * or the multiple attribute" (Section 4.7.15 of the JSR-283 draft specification). *

*/ - private final Multimap multiValuedPropertyDefinitions = new LinkedListMultimap(); + private final Multimap multiValuedPropertyDefinitions = LinkedListMultimap.create(); /** * A local cache of all defined and inherited property definitions, keyed by their name, that allow single values. This * includes residual property definitions, which are keyed by the {@link JcrNodeType#RESIDUAL_NAME}. The content of this map @@ -92,10 +92,10 @@ * or the multiple attribute" (Section 4.7.15 of the JSR-283 draft specification). *

*/ - private final Multimap singleValuedPropertyDefinitions = new LinkedListMultimap(); + private final Multimap singleValuedPropertyDefinitions = LinkedListMultimap.create(); - private final Multimap allChildNodeDefinitions = new LinkedListMultimap(); - private final Multimap allPropertyDefinitions = new LinkedListMultimap(); + private final Multimap allChildNodeDefinitions = LinkedListMultimap.create(); + private final Multimap allPropertyDefinitions = LinkedListMultimap.create(); DefinitionCache( JcrNodeType nodeType ) { addDefinitionsForTypeAndAllSupertypes(nodeType);