8179930: jdk.test.lib.artifacts.ArtifactResolver::resolve should return Map instead of HashMap
authoriignatyev
Thu, 11 May 2017 16:35:01 -0700
changeset 46215 d5e1d3d5e004
parent 46214 b6b866fd21b2
child 46217 191008ef907f
8179930: jdk.test.lib.artifacts.ArtifactResolver::resolve should return Map instead of HashMap Reviewed-by: mseledtsov, gtriantafill, vlivanov
test/lib/jdk/test/lib/artifacts/ArtifactResolver.java
--- a/test/lib/jdk/test/lib/artifacts/ArtifactResolver.java	Fri May 05 17:48:43 2017 +0200
+++ b/test/lib/jdk/test/lib/artifacts/ArtifactResolver.java	Thu May 11 16:35:01 2017 -0700
@@ -26,9 +26,10 @@
 import java.io.FileNotFoundException;
 import java.nio.file.Path;
 import java.util.HashMap;
+import java.util.Map;
 
 public class ArtifactResolver {
-    public static HashMap<String, Path> resolve(Class klass) throws FileNotFoundException {
+    public static Map<String, Path> resolve(Class<?> klass) throws FileNotFoundException {
         ArtifactManager manager = new DefaultArtifactManager();
         try {
             String managerName = System.getProperty("jdk.test.lib.artifacts.artifactmanager");
@@ -41,12 +42,12 @@
             // If we end up here, we'll use the DefaultArtifactManager
         }
 
-        ArtifactContainer artifactContainer = (ArtifactContainer) klass.getAnnotation(ArtifactContainer.class);
+        ArtifactContainer artifactContainer = klass.getAnnotation(ArtifactContainer.class);
         HashMap<String, Path> locations = new HashMap<>();
         Artifact[] artifacts;
 
         if (artifactContainer == null) {
-            artifacts = new Artifact[]{(Artifact) klass.getAnnotation(Artifact.class)};
+            artifacts = new Artifact[]{klass.getAnnotation(Artifact.class)};
         } else {
             artifacts = artifactContainer.value();
         }