[jbosstools-commits] JBoss Tools SVN: r34841 - trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition.

Author: dgolovin Date: 2011-09-16 20:18:03 -0400 (Fri, 16 Sep 2011) New Revision: 34841 Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/AbstractTypeDefinition.java Log: code clean up; return Collection.emptySet() instead of cretaing new empty Set instance Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/AbstractTypeDefinition.java =================================================================== --- trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/AbstractTypeDefinition.java 2011-09-17 00:06:02 UTC (rev 34840) +++ trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/AbstractTypeDefinition.java 2011-09-17 00:18:03 UTC (rev 34841) @@ -10,6 +10,7 @@ ******************************************************************************/ package org.jboss.tools.cdi.internal.core.impl.definition; +import java.util.Collections; import java.util.HashMap; import java.util.HashSet; import java.util.Map; @@ -92,11 +93,17 @@ } public Set<IParametedType> getInheritedTypes() { - return parametedType == null ? new HashSet<IParametedType>() : parametedType.getInheritedTypes(); + if(parametedType == null) { + return Collections.emptySet(); + } + return parametedType.getInheritedTypes(); } public Set<IParametedType> getAllTypes() { - return parametedType == null ? new HashSet<IParametedType>() : parametedType.getAllTypes(); + if(parametedType == null) { + return Collections.emptySet(); + } + return parametedType.getAllTypes(); } /**