Author: fbricon Date: 2012-07-06 08:57:02 -0400 (Fri, 06 Jul 2012) New Revision: 42439 Modified: trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java Log: Remove unnecessary loop Modified: trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java =================================================================== --- trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java 2012-07-06 12:10:17 UTC (rev 42438) +++ trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java 2012-07-06 12:57:02 UTC (rev 42439) @@ -86,23 +86,18 @@ } private boolean isHibernateProject(MavenProject mavenProject) { - List<Artifact> artifacts = new ArrayList<Artifact>(); - ArtifactFilter filter = new ScopeArtifactFilter( - Artifact.SCOPE_TEST); + ArtifactFilter filter = new ScopeArtifactFilter(Artifact.SCOPE_TEST); for (Artifact artifact : mavenProject.getArtifacts()) { if (filter.include(artifact)) { - artifacts.add(artifact); + String groupId = artifact.getGroupId(); + if (HIBERNATE_GROUP_ID.equals(groupId)) { + String artifactId = artifact.getArtifactId(); + if (artifactId != null && artifactId.startsWith(HIBERNATE_ARTIFACT_ID_PREFIX)) { + return true; + } + } } } - for (Artifact artifact:artifacts) { - String groupId = artifact.getGroupId(); - if (HIBERNATE_GROUP_ID.equals(groupId)) { - String artifactId = artifact.getArtifactId(); - if (artifactId != null && artifactId.startsWith(HIBERNATE_ARTIFACT_ID_PREFIX)) { - return true; - } - } - } return false; } }