More Java5 like code usage.
authorKarl Heinz Marbaise <khmarbaise@apache.org>
Mon, 7 Dec 2015 20:38:51 +0000 (20:38 +0000)
committerKarl Heinz Marbaise <khmarbaise@apache.org>
Mon, 7 Dec 2015 20:38:51 +0000 (20:38 +0000)
git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1718456 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/maven/shared/io/download/DefaultDownloadManager.java
src/main/java/org/apache/maven/shared/io/download/DownloadManager.java

index 76afe9d..7532a55 100644 (file)
@@ -49,7 +49,7 @@ public class DefaultDownloadManager
 
     private WagonManager wagonManager;
 
-    private Map cache = new HashMap();
+    private Map<String, File> cache = new HashMap<String, File>();
 
     public DefaultDownloadManager()
     {
@@ -63,10 +63,10 @@ public class DefaultDownloadManager
     public File download( String url, MessageHolder messageHolder )
         throws DownloadFailedException
     {
-        return download( url, Collections.EMPTY_LIST, messageHolder );
+        return download( url, Collections.<TransferListener>emptyList(), messageHolder );
     }
 
-    public File download( String url, List transferListeners, MessageHolder messageHolder )
+    public File download( String url, List<TransferListener> transferListeners, MessageHolder messageHolder )
         throws DownloadFailedException
     {
         File downloaded = (File) cache.get( url );
@@ -123,10 +123,9 @@ public class DefaultDownloadManager
         String remotePath = sourceUrl.getPath();
         String baseUrl = url.substring( 0, url.length() - remotePath.length() );
 
-        for ( Iterator it = transferListeners.iterator(); it.hasNext(); )
+        for ( Iterator<TransferListener> it = transferListeners.iterator(); it.hasNext(); )
         {
-            TransferListener listener = (TransferListener) it.next();
-            wagon.addTransferListener( listener );
+            wagon.addTransferListener( it.next() );
         }
 
         // connect to the remote site, and retrieve the archive. Note the separate methods in which
@@ -137,8 +136,8 @@ public class DefaultDownloadManager
 
         try
         {
-            wagon.connect( repo, wagonManager.getAuthenticationInfo( repo.getId() ), wagonManager.getProxy( sourceUrl
-                .getProtocol() ) );
+            wagon.connect( repo, wagonManager.getAuthenticationInfo( repo.getId() ),
+                           wagonManager.getProxy( sourceUrl.getProtocol() ) );
         }
         catch ( ConnectionException e )
         {
@@ -188,10 +187,9 @@ public class DefaultDownloadManager
                     messageHolder.addMessage( "Failed to disconnect wagon for: " + url, e );
                 }
 
-                for ( Iterator it = transferListeners.iterator(); it.hasNext(); )
+                for ( Iterator<TransferListener> it = transferListeners.iterator(); it.hasNext(); )
                 {
-                    TransferListener listener = (TransferListener) it.next();
-                    wagon.removeTransferListener( listener );
+                    wagon.removeTransferListener( it.next() );
                 }
             }
         }
index ff69793..8d3b990 100644 (file)
@@ -23,6 +23,7 @@ import java.io.File;
 import java.util.List;
 
 import org.apache.maven.shared.io.logging.MessageHolder;
+import org.apache.maven.wagon.events.TransferListener;
 
 public interface DownloadManager
 {
@@ -31,7 +32,7 @@ public interface DownloadManager
     File download( String url, MessageHolder messageHolder )
         throws DownloadFailedException;
 
-    File download( String url, List transferListeners, MessageHolder messageHolder )
+    File download( String url, List<TransferListener> transferListeners, MessageHolder messageHolder )
         throws DownloadFailedException;
 
 }
\ No newline at end of file