You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2015/12/19 12:42:42 UTC

svn commit: r1720902 - in /maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact: ./ collect/internal/ deploy/ deploy/internal/ install/ install/internal/ repository/ repository/internal/ resolve/ resolve/internal/

Author: khmarbaise
Date: Sat Dec 19 11:42:41 2015
New Revision: 1720902

URL: http://svn.apache.org/viewvc?rev=1720902&view=rev
Log:
Cleaning up checkstyle reported errors/warnings

Modified:
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/DefaultArtifactCoordinate.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/TransferUtils.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30ArtifactRepositoryAdapter.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30CollectorResult.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31ArtifactRepositoryAdapter.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31CollectorResult.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployer.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployerException.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven30ArtifactDeployer.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven31ArtifactDeployer.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstaller.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstallerException.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/DefaultArtifactInstaller.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Invoker.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven30ArtifactInstaller.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven31ArtifactInstaller.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManager.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManagerException.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/DefaultRepositoryManager.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Invoker.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven30RepositoryManager.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven31RepositoryManager.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolver.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolverException.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResult.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/DefaultArtifactResolver.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Invoker.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResolver.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResult.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResolver.java
    maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResult.java

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/DefaultArtifactCoordinate.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/DefaultArtifactCoordinate.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/DefaultArtifactCoordinate.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/DefaultArtifactCoordinate.java Sat Dec 19 11:42:41 2015
@@ -49,49 +49,49 @@ package org.apache.maven.shared.artifact
  * 
  * public void setGroupId( String groupId )
  * {
- *   this.artifact.setGroupId( groupId );
+ *     this.artifact.setGroupId( groupId );
  * }
  * 
  * public void setArtifactId( String artifactId )
  * {
- *   this.artifact.setArtifactId( artifactId );
+ *     this.artifact.setArtifactId( artifactId );
  * }
  * 
  * public void setVersion( String version )
  * {
- *   this.artifact.setVersion( version );
+ *     this.artifact.setVersion( version );
  * }
  * 
  * public void setClassifier( String classifier )
  * {
- *   this.artifact.setClassifier( classifier );
+ *     this.artifact.setClassifier( classifier );
  * }
  * 
  * public void setType( String type )
  * {
- *   this.artifact.setType( type );
+ *     this.artifact.setType( type );
  * }
  * </pre>
  * 
- * <strong>Note: </strong> type is not the same as extension! 
- * {@link org.apache.maven.artifact.handler.ArtifactHandler}s are used to map a type to an extension.  
- * 
+ * <strong>Note: </strong> type is not the same as extension! {@link org.apache.maven.artifact.handler.ArtifactHandler}s
+ * are used to map a type to an extension.
  * 
  * @author Robert Scholte
  * @since 3.0
  */
-public class DefaultArtifactCoordinate implements ArtifactCoordinate
+public class DefaultArtifactCoordinate
+    implements ArtifactCoordinate
 {
     private String groupId;
-    
+
     private String artifactId;
-    
+
     private String version;
-    
+
     private String type;
-    
+
     private String classifier;
-    
+
     @Override
     public final String getGroupId()
     {
@@ -161,7 +161,7 @@ public class DefaultArtifactCoordinate i
     {
         this.classifier = classifier;
     }
-    
+
     /**
      * @see org.apache.maven.artifact.DefaultArtifact#toString()
      */
@@ -170,15 +170,15 @@ public class DefaultArtifactCoordinate i
     {
         StringBuilder sb =
             new StringBuilder().append( groupId ).append( ':' ).append( artifactId ).append( ':' ).append( getType() );
-        
+
         if ( classifier != null )
         {
             sb.append( ':' ).append( classifier );
         }
-        
+
         sb.append( ':' ).append( version );
-        
+
         return sb.toString();
     }
-    
+
 }

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/TransferUtils.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/TransferUtils.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/TransferUtils.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/TransferUtils.java Sat Dec 19 11:42:41 2015
@@ -30,7 +30,6 @@ import org.apache.maven.project.MavenPro
  * Utility class
  * 
  * @author Robert Scholte
- *
  */
 public final class TransferUtils
 {
@@ -45,13 +44,13 @@ public final class TransferUtils
     public static ArtifactCoordinate toArtifactCoordinate( Artifact artifact )
     {
         DefaultArtifactCoordinate coordinate = new DefaultArtifactCoordinate();
-        
+
         coordinate.setGroupId( artifact.getGroupId() );
         coordinate.setArtifactId( artifact.getArtifactId() );
         coordinate.setVersion( artifact.getVersion() );
         coordinate.setType( artifact.getType() );
         coordinate.setClassifier( artifact.getClassifier() );
-        
+
         return coordinate;
     }
 
@@ -62,13 +61,13 @@ public final class TransferUtils
     public static ArtifactCoordinate toArtifactCoordinate( Dependency dependency )
     {
         DefaultArtifactCoordinate coordinate = new DefaultArtifactCoordinate();
-        
+
         coordinate.setGroupId( dependency.getGroupId() );
         coordinate.setArtifactId( dependency.getArtifactId() );
         coordinate.setVersion( dependency.getVersion() );
         coordinate.setType( dependency.getType() );
         coordinate.setClassifier( dependency.getClassifier() );
-        
+
         return coordinate;
     }
 
@@ -79,11 +78,11 @@ public final class TransferUtils
     public static ArtifactCoordinate toArtifactCoordinate( Extension extension )
     {
         DefaultArtifactCoordinate coordinate = new DefaultArtifactCoordinate();
-        
+
         coordinate.setGroupId( extension.getGroupId() );
         coordinate.setArtifactId( extension.getArtifactId() );
         coordinate.setVersion( extension.getVersion() );
-        
+
         return coordinate;
     }
 
@@ -94,12 +93,12 @@ public final class TransferUtils
     public static ArtifactCoordinate toArtifactCoordinate( MavenProject project )
     {
         DefaultArtifactCoordinate coordinate = new DefaultArtifactCoordinate();
-        
+
         coordinate.setGroupId( project.getGroupId() );
         coordinate.setArtifactId( project.getArtifactId() );
         coordinate.setVersion( project.getVersion() );
         coordinate.setType( project.getPackaging() );
-        
+
         return coordinate;
     }
 
@@ -110,12 +109,12 @@ public final class TransferUtils
     public static ArtifactCoordinate toArtifactCoordinate( Parent parent )
     {
         DefaultArtifactCoordinate coordinate = new DefaultArtifactCoordinate();
-        
+
         coordinate.setGroupId( parent.getGroupId() );
         coordinate.setArtifactId( parent.getArtifactId() );
         coordinate.setVersion( parent.getVersion() );
         coordinate.setType( "pom" );
-        
+
         return coordinate;
     }
 
@@ -126,15 +125,15 @@ public final class TransferUtils
     public static ArtifactCoordinate toArtifactCoordinate( Plugin plugin )
     {
         DefaultArtifactCoordinate coordinate = new DefaultArtifactCoordinate();
-        
+
         coordinate.setGroupId( plugin.getGroupId() );
         coordinate.setArtifactId( plugin.getArtifactId() );
         coordinate.setVersion( plugin.getVersion() );
         coordinate.setType( "maven-plugin" );
-        
+
         return coordinate;
     }
-    
+
     /**
      * @param artifact {@link Artifact}
      * @return {@link Dependency}
@@ -142,14 +141,14 @@ public final class TransferUtils
     public static Dependency toDependency( Artifact artifact )
     {
         Dependency dependency = new Dependency();
-        
+
         dependency.setGroupId( artifact.getGroupId() );
         dependency.setArtifactId( artifact.getArtifactId() );
         dependency.setVersion( artifact.getVersion() );
         dependency.setType( artifact.getType() );
         dependency.setClassifier( artifact.getClassifier() );
         dependency.setScope( artifact.getScope() );
-        
+
         return dependency;
     }
 
@@ -160,13 +159,13 @@ public final class TransferUtils
     public static Dependency toDependency( ArtifactCoordinate coordinate )
     {
         Dependency dependency = new Dependency();
-        
+
         dependency.setGroupId( coordinate.getGroupId() );
         dependency.setArtifactId( coordinate.getArtifactId() );
         dependency.setVersion( coordinate.getVersion() );
         dependency.setType( coordinate.getType() );
         dependency.setClassifier( coordinate.getClassifier() );
-        
+
         return dependency;
     }
 

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30ArtifactRepositoryAdapter.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30ArtifactRepositoryAdapter.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30ArtifactRepositoryAdapter.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30ArtifactRepositoryAdapter.java Sat Dec 19 11:42:41 2015
@@ -42,6 +42,9 @@ public class Maven30ArtifactRepositoryAd
     
     private RemoteRepository remoteRepository;
 
+    /**
+     * @param remoteRepository {@link RemoteRepository}
+     */
     public Maven30ArtifactRepositoryAdapter( RemoteRepository remoteRepository )
     {
         this.remoteRepository = remoteRepository;

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30CollectorResult.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30CollectorResult.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30CollectorResult.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven30CollectorResult.java Sat Dec 19 11:42:41 2015
@@ -41,6 +41,9 @@ public class Maven30CollectorResult impl
 {
     private final CollectResult collectResult;
     
+    /**
+     * @param collectResult {@link CollectorResult}
+     */
     public Maven30CollectorResult( CollectResult collectResult )
     {
         this.collectResult = collectResult;

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31ArtifactRepositoryAdapter.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31ArtifactRepositoryAdapter.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31ArtifactRepositoryAdapter.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31ArtifactRepositoryAdapter.java Sat Dec 19 11:42:41 2015
@@ -42,6 +42,9 @@ public class Maven31ArtifactRepositoryAd
     
     private RemoteRepository remoteRepository;
 
+    /**
+     * @param remoteRepository {@link RemoteRepository}
+     */
     public Maven31ArtifactRepositoryAdapter( RemoteRepository remoteRepository )
     {
         this.remoteRepository = remoteRepository;

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31CollectorResult.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31CollectorResult.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31CollectorResult.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/collect/internal/Maven31CollectorResult.java Sat Dec 19 11:42:41 2015
@@ -41,6 +41,9 @@ public class Maven31CollectorResult impl
 {
     private final CollectResult collectResult;
     
+    /**
+     * @param collectResult {@link CollectorResult}
+     */
     public Maven31CollectorResult( CollectResult collectResult )
     {
         this.collectResult = collectResult;

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployer.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployer.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployer.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployer.java Sat Dec 19 11:42:41 2015
@@ -31,15 +31,20 @@ import org.apache.maven.project.ProjectB
 public interface ArtifactDeployer
 {
 
+    /**
+     * @param request {@link ProjectBuildingRequest}
+     * @param mavenArtifacts {@link Artifact}
+     * @throws ArtifactDeployerException in case of an error.
+     */
     void deploy( ProjectBuildingRequest request, Collection<Artifact> mavenArtifacts )
-                    throws ArtifactDeployerException;
+        throws ArtifactDeployerException;
 
     /**
      * @param request the building request
-     * @param remoteRepository the repository to deploy to. If {@code null} the {@code mavenArtifact.getRepository()} 
-     *          is used.
+     * @param remoteRepository the repository to deploy to. If {@code null} the {@code mavenArtifact.getRepository()} is
+     *            used.
      * @param mavenArtifacts the artifacts to deploy
-     * @throws ArtifactDeployerException
+     * @throws ArtifactDeployerException in case of an error.
      */
     void deploy( ProjectBuildingRequest request, ArtifactRepository remoteRepository,
                  Collection<Artifact> mavenArtifacts )

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployerException.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployerException.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployerException.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/ArtifactDeployerException.java Sat Dec 19 11:42:41 2015
@@ -26,6 +26,15 @@ public class ArtifactDeployerException
     extends Exception
 {
 
+    /**
+     * 
+     */
+    private static final long serialVersionUID = 7421964724059077698L;
+
+    /**
+     * @param message The message of the error.
+     * @param e {@link Exception}
+     */
     public ArtifactDeployerException( String message, Exception e )
     {
         super( message, e );

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven30ArtifactDeployer.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven30ArtifactDeployer.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven30ArtifactDeployer.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven30ArtifactDeployer.java Sat Dec 19 11:42:41 2015
@@ -57,45 +57,45 @@ public class Maven30ArtifactDeployer
     {
         deploy( buildingRequest, null, mavenArtifacts );
     }
-    
+
     @Override
     public void deploy( ProjectBuildingRequest buildingRequest, ArtifactRepository remoteRepository,
                         Collection<org.apache.maven.artifact.Artifact> mavenArtifacts )
-        throws ArtifactDeployerException
+                            throws ArtifactDeployerException
     {
         // prepare request
         DeployRequest request = new DeployRequest();
 
         RepositorySystemSession session =
-                        (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
-        
+            (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
+
         RemoteRepository defaultRepository = null;
-        
+
         if ( remoteRepository != null )
         {
             defaultRepository = getRemoteRepository( session, remoteRepository );
         }
-        
+
         // transform artifacts
         for ( org.apache.maven.artifact.Artifact mavenArtifact : mavenArtifacts )
         {
             Artifact aetherArtifact =
-                            (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
-                                                       org.apache.maven.artifact.Artifact.class, mavenArtifact );
+                (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
+                                           org.apache.maven.artifact.Artifact.class, mavenArtifact );
             request.addArtifact( aetherArtifact );
-            
+
             RemoteRepository aetherRepository;
             if ( remoteRepository == null )
             {
-                aetherRepository =  getRemoteRepository( session, mavenArtifact.getRepository() );
+                aetherRepository = getRemoteRepository( session, mavenArtifact.getRepository() );
             }
             else
             {
                 aetherRepository = defaultRepository;
             }
-            
+
             request.setRepository( aetherRepository );
-            
+
             for ( ArtifactMetadata metadata : mavenArtifact.getMetadataList() )
             {
                 if ( metadata instanceof ProjectArtifactMetadata )
@@ -105,7 +105,7 @@ public class Maven30ArtifactDeployer
                     request.addArtifact( pomArtifact );
                 }
                 else if ( // metadata instanceof SnapshotArtifactRepositoryMetadata ||
-                    metadata instanceof ArtifactRepositoryMetadata )
+                metadata instanceof ArtifactRepositoryMetadata )
                 {
                     // eaten, handled by repo system
                 }
@@ -115,7 +115,7 @@ public class Maven30ArtifactDeployer
                 }
             }
         }
-        
+
         // deploy
         try
         {
@@ -135,17 +135,17 @@ public class Maven30ArtifactDeployer
                                                                          org.apache.maven.artifact.repository.ArtifactRepository.class,
                                                                          remoteRepository );
         // CHECKSTYLE_ON: LineLength
-        
+
         if ( aetherRepo.getAuthentication() == null )
         {
             aetherRepo.setAuthentication( session.getAuthenticationSelector().getAuthentication( aetherRepo ) );
         }
-        
+
         if ( aetherRepo.getProxy() == null )
         {
             aetherRepo.setProxy( session.getProxySelector().getProxy( aetherRepo ) );
         }
-        
+
         return aetherRepo;
     }
 }

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven31ArtifactDeployer.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven31ArtifactDeployer.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven31ArtifactDeployer.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/deploy/internal/Maven31ArtifactDeployer.java Sat Dec 19 11:42:41 2015
@@ -43,12 +43,13 @@ import org.eclipse.aether.util.artifact.
  * 
  */
 @Component( role = ArtifactDeployer.class, hint = "maven31" )
-public class Maven31ArtifactDeployer implements ArtifactDeployer
+public class Maven31ArtifactDeployer
+    implements ArtifactDeployer
 {
 
     @Requirement
     private RepositorySystem repositorySystem;
-    
+
     @Override
     public void deploy( ProjectBuildingRequest buildingRequest,
                         Collection<org.apache.maven.artifact.Artifact> mavenArtifacts )
@@ -56,32 +57,33 @@ public class Maven31ArtifactDeployer imp
     {
         deploy( buildingRequest, null, mavenArtifacts );
     }
-    
+
+    /** {@inheritDoc} */
     public void deploy( ProjectBuildingRequest buildingRequest, ArtifactRepository remoteRepository,
                         Collection<org.apache.maven.artifact.Artifact> mavenArtifacts )
-        throws ArtifactDeployerException
+                            throws ArtifactDeployerException
     {
         // prepare request
         DeployRequest request = new DeployRequest();
 
         RepositorySystemSession session =
-                        (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
+            (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
 
         RemoteRepository defaultRepository = null;
-        
+
         if ( remoteRepository != null )
         {
             defaultRepository = getRemoteRepository( session, remoteRepository );
         }
-        
+
         // transform artifacts
         for ( org.apache.maven.artifact.Artifact mavenArtifact : mavenArtifacts )
         {
             Artifact aetherArtifact =
-                            (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
-                                                       org.apache.maven.artifact.Artifact.class, mavenArtifact );
+                (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
+                                           org.apache.maven.artifact.Artifact.class, mavenArtifact );
             request.addArtifact( aetherArtifact );
-            
+
             RemoteRepository aetherRepository;
             if ( remoteRepository == null )
             {
@@ -91,7 +93,7 @@ public class Maven31ArtifactDeployer imp
             {
                 aetherRepository = defaultRepository;
             }
-            
+
             request.setRepository( aetherRepository );
 
             for ( ArtifactMetadata metadata : mavenArtifact.getMetadataList() )
@@ -103,7 +105,7 @@ public class Maven31ArtifactDeployer imp
                     request.addArtifact( pomArtifact );
                 }
                 else if ( // metadata instanceof SnapshotArtifactRepositoryMetadata ||
-                    metadata instanceof ArtifactRepositoryMetadata )
+                metadata instanceof ArtifactRepositoryMetadata )
                 {
                     // eaten, handled by repo system
                 }
@@ -124,30 +126,30 @@ public class Maven31ArtifactDeployer imp
             throw new ArtifactDeployerException( e.getMessage(), e );
         }
     }
-    
+
     private RemoteRepository getRemoteRepository( RepositorySystemSession session, ArtifactRepository remoteRepository )
-                    throws ArtifactDeployerException
+        throws ArtifactDeployerException
     {
         // CHECKSTYLE_OFF: LineLength
         RemoteRepository aetherRepo = (RemoteRepository) Invoker.invoke( RepositoryUtils.class, "toRepo",
                                                                          org.apache.maven.artifact.repository.ArtifactRepository.class,
                                                                          remoteRepository );
         // CHECKSTYLE_ON: LineLength
-        
+
         if ( aetherRepo.getAuthentication() == null || aetherRepo.getProxy() == null )
         {
             RemoteRepository.Builder builder = new RemoteRepository.Builder( aetherRepo );
-            
+
             if ( aetherRepo.getAuthentication() == null )
             {
                 builder.setAuthentication( session.getAuthenticationSelector().getAuthentication( aetherRepo ) );
             }
-            
+
             if ( aetherRepo.getProxy() == null )
             {
                 builder.setProxy( session.getProxySelector().getProxy( aetherRepo ) );
             }
-            
+
             aetherRepo = builder.build();
         }
 

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstaller.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstaller.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstaller.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstaller.java Sat Dec 19 11:42:41 2015
@@ -25,11 +25,16 @@ import org.apache.maven.artifact.Artifac
 import org.apache.maven.project.ProjectBuildingRequest;
 
 /**
- * 
+ * @author Robert Scholte
  */
 public interface ArtifactInstaller
 {
 
+    /**
+     * @param request {@link ProjectBuildingRequest}
+     * @param mavenArtifacts {@link Artifact}
+     * @throws ArtifactInstallerException in case of an error.
+     */
     void install( ProjectBuildingRequest request, Collection<Artifact> mavenArtifacts )
         throws ArtifactInstallerException;
 }

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstallerException.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstallerException.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstallerException.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/ArtifactInstallerException.java Sat Dec 19 11:42:41 2015
@@ -26,6 +26,15 @@ public class ArtifactInstallerException
     extends Exception
 {
 
+    /**
+     * 
+     */
+    private static final long serialVersionUID = 3652561971360586373L;
+
+    /**
+     * @param message The message of the error.
+     * @param e {@link Exception}
+     */
     public ArtifactInstallerException( String message, Exception e )
     {
         super( message, e );

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/DefaultArtifactInstaller.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/DefaultArtifactInstaller.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/DefaultArtifactInstaller.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/DefaultArtifactInstaller.java Sat Dec 19 11:42:41 2015
@@ -43,6 +43,7 @@ public class DefaultArtifactInstaller
 
     private PlexusContainer container;
 
+    /** {@inheritDoc} */
     public void install( ProjectBuildingRequest request, Collection<Artifact> mavenArtifacts )
         throws ArtifactInstallerException
     {
@@ -85,7 +86,7 @@ public class DefaultArtifactInstaller
     /**
      * Injects the Plexus content.
      *
-     * @param context   Plexus context to inject.
+     * @param context Plexus context to inject.
      * @throws ContextException if the PlexusContainer could not be located.
      */
     public void contextualize( Context context )

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Invoker.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Invoker.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Invoker.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Invoker.java Sat Dec 19 11:42:41 2015
@@ -81,9 +81,9 @@ final class Invoker
             throw new ArtifactInstallerException( e.getMessage(), e );
         }
     }
-    
+
     public static Object invoke( Class<?> objectClazz, String staticMethod, Class<?> argClazz, Object arg )
-                    throws ArtifactInstallerException
+        throws ArtifactInstallerException
     {
         try
         {

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven30ArtifactInstaller.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven30ArtifactInstaller.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven30ArtifactInstaller.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven30ArtifactInstaller.java Sat Dec 19 11:42:41 2015
@@ -47,9 +47,10 @@ public class Maven30ArtifactInstaller
     @Requirement
     private RepositorySystem repositorySystem;
 
+    /** {@inheritDoc} */
     public void install( ProjectBuildingRequest buildingRequest,
                          Collection<org.apache.maven.artifact.Artifact> mavenArtifacts )
-        throws ArtifactInstallerException
+                             throws ArtifactInstallerException
     {
         // prepare installRequest
         InstallRequest request = new InstallRequest();
@@ -71,7 +72,7 @@ public class Maven30ArtifactInstaller
                     request.addArtifact( pomArtifact );
                 }
                 else if ( // metadata instanceof SnapshotArtifactRepositoryMetadata ||
-                    metadata instanceof ArtifactRepositoryMetadata )
+                metadata instanceof ArtifactRepositoryMetadata )
                 {
                     // eaten, handled by repo system
                 }

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven31ArtifactInstaller.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven31ArtifactInstaller.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven31ArtifactInstaller.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/install/internal/Maven31ArtifactInstaller.java Sat Dec 19 11:42:41 2015
@@ -41,15 +41,17 @@ import org.eclipse.aether.util.artifact.
  * 
  */
 @Component( role = ArtifactInstaller.class, hint = "maven31" )
-public class Maven31ArtifactInstaller implements ArtifactInstaller 
+public class Maven31ArtifactInstaller
+    implements ArtifactInstaller
 {
 
     @Requirement
     private RepositorySystem repositorySystem;
-    
+
+    /** {@inheritDoc} */
     public void install( ProjectBuildingRequest buildingRequest,
                          Collection<org.apache.maven.artifact.Artifact> mavenArtifacts )
-        throws ArtifactInstallerException
+                             throws ArtifactInstallerException
     {
         // prepare installRequest
         InstallRequest request = new InstallRequest();

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManager.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManager.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManager.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManager.java Sat Dec 19 11:42:41 2015
@@ -31,19 +31,29 @@ import org.apache.maven.shared.artifact.
 public interface RepositoryManager
 {
 
+    /**
+     * @param buildingRequest {@link ProjectBuildingRequest}
+     * @param artifact {@link Artifact}
+     * @return the path of the local artifact.
+     */
     String getPathForLocalArtifact( ProjectBuildingRequest buildingRequest, Artifact artifact );
 
+    /**
+     * @param buildingRequest {@link ProjectBuildingRequest}
+     * @param coordinate {@link ArtifactCoordinate}
+     * @return the path for the local artifact.
+     */
     String getPathForLocalArtifact( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate );
 
     /**
-     * Create a new {@code ProjectBuildingRequest} with an adjusted repository session. 
+     * Create a new {@code ProjectBuildingRequest} with an adjusted repository session.
      * 
      * @param request the project building request
      * @param basedir the base directory of the local repository
      * @return a new project building request
      */
     ProjectBuildingRequest setLocalRepositoryBasedir( ProjectBuildingRequest request, File basedir );
-    
+
     /**
      * Get the localRepositryBasedir as specified in the repository session of the request
      * 

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManagerException.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManagerException.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManagerException.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/RepositoryManagerException.java Sat Dec 19 11:42:41 2015
@@ -22,9 +22,16 @@ package org.apache.maven.shared.artifact
 /**
  * 
  */
-public class RepositoryManagerException extends Exception
+public class RepositoryManagerException
+    extends Exception
 {
 
+    private static final long serialVersionUID = 8396752347704808545L;
+
+    /**
+     * @param message The message of the exception.
+     * @param cause {@link Throwable}
+     */
     public RepositoryManagerException( String message, Throwable cause )
     {
         super( message, cause );

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/DefaultRepositoryManager.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/DefaultRepositoryManager.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/DefaultRepositoryManager.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/DefaultRepositoryManager.java Sat Dec 19 11:42:41 2015
@@ -75,7 +75,7 @@ public class DefaultRepositoryManager
             throw new IllegalStateException( e.getMessage(), e );
         }
     }
-    
+
     @Override
     public ProjectBuildingRequest setLocalRepositoryBasedir( ProjectBuildingRequest request, File basedir )
     {

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Invoker.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Invoker.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Invoker.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Invoker.java Sat Dec 19 11:42:41 2015
@@ -81,9 +81,9 @@ final class Invoker
             throw new RepositoryManagerException( e.getMessage(), e );
         }
     }
-    
+
     public static Object invoke( Class<?> objectClazz, String staticMethod, Class<?> argClazz, Object arg )
-                    throws RepositoryManagerException
+        throws RepositoryManagerException
     {
         try
         {

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven30RepositoryManager.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven30RepositoryManager.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven30RepositoryManager.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven30RepositoryManager.java Sat Dec 19 11:42:41 2015
@@ -51,7 +51,7 @@ public class Maven30RepositoryManager
 
     @Requirement
     private RepositorySystem repositorySystem;
-    
+
     @Requirement
     private ArtifactHandlerManager artifactHandlerManager;
 
@@ -60,15 +60,14 @@ public class Maven30RepositoryManager
                                            org.apache.maven.artifact.Artifact mavenArtifact )
     {
         Artifact aetherArtifact;
-        
+
         RepositorySystemSession session;
-        
+
         // LRM.getPathForLocalArtifact() won't throw an Exception, so translate reflection error to RuntimeException
         try
         {
-            aetherArtifact =
-                (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
-                                           org.apache.maven.artifact.Artifact.class, mavenArtifact );
+            aetherArtifact = (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
+                                                        org.apache.maven.artifact.Artifact.class, mavenArtifact );
 
             session = (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
         }
@@ -79,12 +78,12 @@ public class Maven30RepositoryManager
 
         return session.getLocalRepositoryManager().getPathForLocalArtifact( aetherArtifact );
     }
-    
+
     @Override
     public String getPathForLocalArtifact( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate )
     {
         Artifact aetherArtifact;
-        
+
         RepositorySystemSession session;
 
         // LRM.getPathForLocalArtifact() won't throw an Exception, so translate reflection error to RuntimeException
@@ -93,16 +92,16 @@ public class Maven30RepositoryManager
             ArtifactTypeRegistry typeRegistry =
                 (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
                                                        ArtifactHandlerManager.class, artifactHandlerManager );
-            
+
             aetherArtifact = toArtifact( coordinate, typeRegistry );
-            
+
             session = (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
         }
         catch ( RepositoryManagerException e )
         {
             throw new RuntimeException( e.getMessage(), e );
         }
-       
+
         return session.getLocalRepositoryManager().getPathForLocalArtifact( aetherArtifact );
     }
 
@@ -146,7 +145,7 @@ public class Maven30RepositoryManager
 
         return newRequest;
     }
-    
+
     @Override
     public File getLocalRepositoryBasedir( ProjectBuildingRequest buildingRequest )
     {
@@ -162,11 +161,20 @@ public class Maven30RepositoryManager
         return session.getLocalRepository().getBasedir();
     }
 
+    /**
+     * @param localRepository {@link LocalRepository}
+     * @return the resolved type.
+     */
     protected String resolveRepositoryType( LocalRepository localRepository )
     {
         return localRepository.getContentType();
     }
-    
+
+    /**
+     * @param coordinate {@link ArtifactCoordinate}
+     * @param typeRegistry {link ArtifactType}
+     * @return {@link Artifact}
+     */
     protected Artifact toArtifact( ArtifactCoordinate coordinate, ArtifactTypeRegistry typeRegistry )
     {
         if ( coordinate == null )

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven31RepositoryManager.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven31RepositoryManager.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven31RepositoryManager.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/repository/internal/Maven31RepositoryManager.java Sat Dec 19 11:42:41 2015
@@ -60,13 +60,12 @@ public class Maven31RepositoryManager
     {
         Artifact aetherArtifact;
         RepositorySystemSession session;
-        
+
         // LRM.getPathForLocalArtifact() won't throw an Exception, so translate reflection error to RuntimeException
         try
         {
-            aetherArtifact =
-                (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
-                                           org.apache.maven.artifact.Artifact.class, mavenArtifact );
+            aetherArtifact = (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
+                                                        org.apache.maven.artifact.Artifact.class, mavenArtifact );
 
             session = (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
         }
@@ -77,12 +76,12 @@ public class Maven31RepositoryManager
 
         return session.getLocalRepositoryManager().getPathForLocalArtifact( aetherArtifact );
     }
-    
+
     @Override
     public String getPathForLocalArtifact( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate )
     {
         Artifact aetherArtifact;
-        
+
         RepositorySystemSession session;
 
         // LRM.getPathForLocalArtifact() won't throw an Exception, so translate reflection error to RuntimeException
@@ -91,16 +90,16 @@ public class Maven31RepositoryManager
             ArtifactTypeRegistry typeRegistry =
                 (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
                                                        ArtifactHandlerManager.class, artifactHandlerManager );
-            
+
             aetherArtifact = toArtifact( coordinate, typeRegistry );
-            
+
             session = (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
         }
         catch ( RepositoryManagerException e )
         {
             throw new RuntimeException( e.getMessage(), e );
         }
-       
+
         return session.getLocalRepositoryManager().getPathForLocalArtifact( aetherArtifact );
     }
 
@@ -161,6 +160,10 @@ public class Maven31RepositoryManager
         return session.getLocalRepository().getBasedir();
     }
 
+    /**
+     * @param localRepository {@link LocalRepository}
+     * @return the resolved type.
+     */
     protected String resolveRepositoryType( LocalRepository localRepository )
     {
         String repositoryType;
@@ -175,7 +178,12 @@ public class Maven31RepositoryManager
         }
         return repositoryType;
     }
-    
+
+    /**
+     * @param coordinate {@link ArtifactCoordinate}
+     * @param typeRegistry {@link ArtifactType}
+     * @return {@link Artifact}
+     */
     protected Artifact toArtifact( ArtifactCoordinate coordinate, ArtifactTypeRegistry typeRegistry )
     {
         if ( coordinate == null )

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolver.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolver.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolver.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolver.java Sat Dec 19 11:42:41 2015
@@ -33,40 +33,51 @@ import org.apache.maven.shared.artifact.
 public interface ArtifactResolver
 {
 
-    ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest , Artifact mavenArtifact )
+    /**
+     * @param buildingRequest {@link ProjectBuildingRequest}
+     * @param mavenArtifact {@link Artifact}
+     * @return {@link ArtifactResult}
+     * @throws ArtifactResolverException in case of an error.
+     */
+    ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest, Artifact mavenArtifact )
         throws ArtifactResolverException;
 
-    ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest , ArtifactCoordinate coordinate )
-                    throws ArtifactResolverException;
+    /**
+     * @param buildingRequest {@link ProjectBuildingRequest}
+     * @param coordinate {@link ArtifactCoordinate}
+     * @return {@link ArtifactResult}
+     * @throws ArtifactResolverException in case of an error.
+     */
+    ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate )
+        throws ArtifactResolverException;
 
     /**
-     * This will resolve the dependencies of the coordinate, not resolving the the artifact of the coordinate itself.
-     * If the coordinate needs to be resolved too, use 
-     * {@link #resolveDependencies(ProjectBuildingRequest, Collection, Collection, TransformableFilter)} passing 
+     * This will resolve the dependencies of the coordinate, not resolving the the artifact of the coordinate itself. If
+     * the coordinate needs to be resolved too, use
+     * {@link #resolveDependencies(ProjectBuildingRequest, Collection, Collection, TransformableFilter)} passing
      * {@code Collections.singletonList(coordinate)}
      * 
-     * @param buildingRequest
-     * @param coordinate
-     * @param filter
-     * @return
-     * @throws ArtifactResolverException
+     * @param buildingRequest {@link ProjectBuildingRequest}
+     * @param coordinate {@link ArtifactCoordinate}
+     * @param filter {@link TransformableFilter}
+     * @return the resolved dependencies.
+     * @throws ArtifactResolverException in case of an error.
      */
-    Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
-                                                  ArtifactCoordinate coordinate, TransformableFilter filter )
-        throws ArtifactResolverException;
+    Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate,
+                                                  TransformableFilter filter )
+                                                      throws ArtifactResolverException;
 
     /**
-     * 
      * @param buildingRequest the project building request, never {@code null}
      * @param dependencies the dependencies to resolve, never {@code null}
      * @param managedDependencies managed dependencies, can be {@code null}
      * @param filter a filter, can be {@code null}
-     * @return
-     * @throws ArtifactResolverException
+     * @return the resolved dependencies.
+     * @throws ArtifactResolverException in case of an error.
      */
     Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
                                                   Collection<Dependency> dependencies,
                                                   Collection<Dependency> managedDependencies,
                                                   TransformableFilter filter )
-                    throws ArtifactResolverException;
+                                                      throws ArtifactResolverException;
 }

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolverException.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolverException.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolverException.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResolverException.java Sat Dec 19 11:42:41 2015
@@ -22,9 +22,16 @@ package org.apache.maven.shared.artifact
 /**
  * 
  */
-public class ArtifactResolverException extends Exception
+public class ArtifactResolverException
+    extends Exception
 {
 
+    private static final long serialVersionUID = 7252294837746943917L;
+
+    /**
+     * @param message The message for the exception.
+     * @param e The exception itself.
+     */
     public ArtifactResolverException( String message, Exception e )
     {
         super( message, e );

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResult.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResult.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResult.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/ArtifactResult.java Sat Dec 19 11:42:41 2015
@@ -29,5 +29,8 @@ import org.apache.maven.artifact.Artifac
  */
 public interface ArtifactResult
 {
+    /**
+     * @return {@link Artifact}
+     */
     Artifact getArtifact();
 }

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/DefaultArtifactResolver.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/DefaultArtifactResolver.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/DefaultArtifactResolver.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/DefaultArtifactResolver.java Sat Dec 19 11:42:41 2015
@@ -47,7 +47,7 @@ public class DefaultArtifactResolver
     private PlexusContainer container;
 
     @Override
-    public ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest , Artifact mavenArtifact  )
+    public ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest, Artifact mavenArtifact )
         throws ArtifactResolverException
     {
         try
@@ -63,7 +63,7 @@ public class DefaultArtifactResolver
             throw new ArtifactResolverException( e.getMessage(), e );
         }
     }
-    
+
     @Override
     public ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest, ArtifactCoordinate coordinate )
         throws ArtifactResolverException
@@ -81,13 +81,13 @@ public class DefaultArtifactResolver
             throw new ArtifactResolverException( e.getMessage(), e );
         }
     }
-    
+
     @Override
     public Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
                                                          Collection<Dependency> coordinates,
                                                          Collection<Dependency> managedDependencies,
                                                          TransformableFilter filter )
-        throws ArtifactResolverException
+                                                             throws ArtifactResolverException
     {
         try
         {
@@ -102,11 +102,11 @@ public class DefaultArtifactResolver
             throw new ArtifactResolverException( e.getMessage(), e );
         }
     }
-    
+
     @Override
     public Iterable<ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
                                                          ArtifactCoordinate coordinate, TransformableFilter filter )
-        throws ArtifactResolverException
+                                                             throws ArtifactResolverException
     {
         try
         {
@@ -121,7 +121,7 @@ public class DefaultArtifactResolver
             throw new ArtifactResolverException( e.getMessage(), e );
         }
     }
-    
+
     /**
      * @return true if the current Maven version is Maven 3.1.
      */

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Invoker.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Invoker.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Invoker.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Invoker.java Sat Dec 19 11:42:41 2015
@@ -81,9 +81,9 @@ final class Invoker
             throw new ArtifactResolverException( e.getMessage(), e );
         }
     }
-    
+
     public static Object invoke( Class<?> objectClazz, String staticMethod, Class<?> argClazz, Object arg )
-                    throws ArtifactResolverException
+        throws ArtifactResolverException
     {
         try
         {
@@ -102,9 +102,9 @@ final class Invoker
             throw new ArtifactResolverException( e.getMessage(), e );
         }
     }
-    
+
     /**
-     * <strong>Note:</strong> Ensure that argClasses and args have the same number of elements 
+     * <strong>Note:</strong> Ensure that argClasses and args have the same number of elements
      * 
      * @param objectClazz the class of the static method
      * @param staticMethod the static method to call
@@ -114,7 +114,7 @@ final class Invoker
      * @throws ArtifactResolverException if any checked exception occurs
      */
     public static Object invoke( Class<?> objectClazz, String staticMethod, Class<?>[] argClasses, Object[] args )
-                    throws ArtifactResolverException
+        throws ArtifactResolverException
     {
         try
         {

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResolver.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResolver.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResolver.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResolver.java Sat Dec 19 11:42:41 2015
@@ -70,12 +70,11 @@ public class Maven30ArtifactResolver
     // CHECKSTYLE_OFF: LineLength
     public org.apache.maven.shared.artifact.resolve.ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest,
                                                                                     org.apache.maven.artifact.Artifact mavenArtifact )
+                                                                                        throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
-        Artifact aetherArtifact =
-            (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact", org.apache.maven.artifact.Artifact.class,
-                                       mavenArtifact );
+        Artifact aetherArtifact = (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
+                                                             org.apache.maven.artifact.Artifact.class, mavenArtifact );
 
         return resolveArtifact( buildingRequest, aetherArtifact );
     }
@@ -84,8 +83,8 @@ public class Maven30ArtifactResolver
     // CHECKSTYLE_OFF: LineLength
     public org.apache.maven.shared.artifact.resolve.ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest,
                                                                                     ArtifactCoordinate coordinate )
+                                                                                        throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         ArtifactTypeRegistry typeRegistry =
             (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
@@ -99,8 +98,8 @@ public class Maven30ArtifactResolver
     // CHECKSTYLE_OFF: LineLength
     private org.apache.maven.shared.artifact.resolve.ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest,
                                                                                      Artifact aetherArtifact )
+                                                                                         throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         @SuppressWarnings( "unchecked" )
         List<RemoteRepository> aetherRepositories =
@@ -138,8 +137,9 @@ public class Maven30ArtifactResolver
     public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
                                                                                                   ArtifactCoordinate coordinate,
                                                                                                   TransformableFilter dependencyFilter )
-    // CHECKSTYLE_ON: LineLength
-          throws ArtifactResolverException
+                                                                                                      // CHECKSTYLE_ON:
+                                                                                                      // LineLength
+                                                                                                      throws ArtifactResolverException
     {
         ArtifactTypeRegistry typeRegistry =
             (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
@@ -163,8 +163,9 @@ public class Maven30ArtifactResolver
                                                                                                   Collection<org.apache.maven.model.Dependency> mavenDependencies,
                                                                                                   Collection<org.apache.maven.model.Dependency> managedMavenDependencies,
                                                                                                   TransformableFilter filter )
-    // CHECKSTYLE_ON: LineLength                                                                                                  
-        throws ArtifactResolverException
+                                                                                                      // CHECKSTYLE_ON:
+                                                                                                      // LineLength
+                                                                                                      throws ArtifactResolverException
     {
         ArtifactTypeRegistry typeRegistry =
             (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
@@ -202,7 +203,8 @@ public class Maven30ArtifactResolver
             (List<RemoteRepository>) Invoker.invoke( RepositoryUtils.class, "toRepos", List.class,
                                                      buildingRequest.getRemoteRepositories() );
 
-        CollectRequest request = new CollectRequest( aetherDependencies, aetherManagedDependencies, aetherRepositories );
+        CollectRequest request =
+            new CollectRequest( aetherDependencies, aetherManagedDependencies, aetherRepositories );
 
         return resolveDependencies( buildingRequest, aetherRepositories, filter, request );
     }
@@ -212,8 +214,8 @@ public class Maven30ArtifactResolver
                                                                                                    List<RemoteRepository> aetherRepositories,
                                                                                                    TransformableFilter dependencyFilter,
                                                                                                    CollectRequest request )
-    // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
+                                                                                                       throws ArtifactResolverException
+    // CHECKSTYLE_ON :LineLength
     {
         try
         {
@@ -226,8 +228,8 @@ public class Maven30ArtifactResolver
             RepositorySystemSession session =
                 (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" );
 
-            List<ArtifactResult> dependencyResults = 
-                            repositorySystem.resolveDependencies( session, request, depFilter );
+            List<ArtifactResult> dependencyResults =
+                repositorySystem.resolveDependencies( session, request, depFilter );
 
             Collection<ArtifactRequest> artifactRequests = new ArrayList<ArtifactRequest>( dependencyResults.size() );
 
@@ -245,7 +247,7 @@ public class Maven30ArtifactResolver
                 public Iterator<org.apache.maven.shared.artifact.resolve.ArtifactResult> iterator()
                 {
                     Collection<org.apache.maven.shared.artifact.resolve.ArtifactResult> artResults =
-                       new ArrayList<org.apache.maven.shared.artifact.resolve.ArtifactResult>( artifactResults.size() );
+                        new ArrayList<org.apache.maven.shared.artifact.resolve.ArtifactResult>( artifactResults.size() );
 
                     for ( ArtifactResult artifactResult : artifactResults )
                     {

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResult.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResult.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResult.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven30ArtifactResult.java Sat Dec 19 11:42:41 2015
@@ -30,15 +30,19 @@ import org.sonatype.aether.resolution.Ar
  * @author Robert Scholte
  * @since 3.0
  */
-public class Maven30ArtifactResult implements org.apache.maven.shared.artifact.resolve.ArtifactResult
+public class Maven30ArtifactResult
+    implements org.apache.maven.shared.artifact.resolve.ArtifactResult
 {
     private final ArtifactResult artifactResult;
 
+    /**
+     * @param artifactResult {@link ArtifactResult}
+     */
     public Maven30ArtifactResult( ArtifactResult artifactResult )
     {
         this.artifactResult = artifactResult;
     }
-    
+
     @Override
     public org.apache.maven.artifact.Artifact getArtifact()
     {

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResolver.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResolver.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResolver.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResolver.java Sat Dec 19 11:42:41 2015
@@ -64,7 +64,7 @@ public class Maven31ArtifactResolver
 {
     @Requirement
     private RepositorySystem repositorySystem;
-    
+
     @Requirement
     private ArtifactHandlerManager artifactHandlerManager;
 
@@ -72,37 +72,36 @@ public class Maven31ArtifactResolver
     // CHECKSTYLE_OFF: LineLength
     public org.apache.maven.shared.artifact.resolve.ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest,
                                                                                     org.apache.maven.artifact.Artifact mavenArtifact )
+                                                                                        throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
-        Artifact aetherArtifact =
-            (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact", org.apache.maven.artifact.Artifact.class,
-                                       mavenArtifact );
+        Artifact aetherArtifact = (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact",
+                                                             org.apache.maven.artifact.Artifact.class, mavenArtifact );
 
         return resolveArtifact( buildingRequest, aetherArtifact );
     }
-    
+
     @Override
     // CHECKSTYLE_OFF: LineLength
     public org.apache.maven.shared.artifact.resolve.ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest,
                                                                                     ArtifactCoordinate coordinate )
+                                                                                        throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         ArtifactTypeRegistry typeRegistry =
-                        (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
-                                                               ArtifactHandlerManager.class, artifactHandlerManager );
+            (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
+                                                   ArtifactHandlerManager.class, artifactHandlerManager );
 
         Dependency aetherDependency = toDependency( coordinate, typeRegistry );
-        
+
         return resolveArtifact( buildingRequest, aetherDependency.getArtifact() );
     }
-    
+
     // CHECKSTYLE_OFF: LineLength
     private org.apache.maven.shared.artifact.resolve.ArtifactResult resolveArtifact( ProjectBuildingRequest buildingRequest,
                                                                                      Artifact aetherArtifact )
+                                                                                         throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         @SuppressWarnings( "unchecked" )
         List<RemoteRepository> aetherRepositories =
@@ -140,15 +139,15 @@ public class Maven31ArtifactResolver
     public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
                                                                                                   ArtifactCoordinate coordinate,
                                                                                                   TransformableFilter dependencyFilter )
+                                                                                                      throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         ArtifactTypeRegistry typeRegistry =
-                        (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
-                                                               ArtifactHandlerManager.class, artifactHandlerManager );
+            (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
+                                                   ArtifactHandlerManager.class, artifactHandlerManager );
 
         Dependency aetherRoot = toDependency( coordinate, typeRegistry );
-        
+
         @SuppressWarnings( "unchecked" )
         List<RemoteRepository> aetherRepositories =
             (List<RemoteRepository>) Invoker.invoke( RepositoryUtils.class, "toRepos", List.class,
@@ -159,16 +158,14 @@ public class Maven31ArtifactResolver
         return resolveDependencies( buildingRequest, aetherRepositories, dependencyFilter, request );
     }
 
-    
-    
     @Override
     // CHECKSTYLE_OFF: LineLength
     public Iterable<org.apache.maven.shared.artifact.resolve.ArtifactResult> resolveDependencies( ProjectBuildingRequest buildingRequest,
                                                                                                   Collection<org.apache.maven.model.Dependency> mavenDependencies,
                                                                                                   Collection<org.apache.maven.model.Dependency> managedMavenDependencies,
                                                                                                   TransformableFilter filter )
+                                                                                                      throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         ArtifactTypeRegistry typeRegistry =
             (ArtifactTypeRegistry) Invoker.invoke( RepositoryUtils.class, "newArtifactTypeRegistry",
@@ -216,8 +213,8 @@ public class Maven31ArtifactResolver
                                                                                                    List<RemoteRepository> aetherRepositories,
                                                                                                    TransformableFilter dependencyFilter,
                                                                                                    CollectRequest request )
+                                                                                                       throws ArtifactResolverException
     // CHECKSTYLE_ON: LineLength
-        throws ArtifactResolverException
     {
         try
         {
@@ -250,8 +247,10 @@ public class Maven31ArtifactResolver
                 @Override
                 public Iterator<org.apache.maven.shared.artifact.resolve.ArtifactResult> iterator()
                 {
+                    // CHECKSTYLE_OFF: LineLength
                     Collection<org.apache.maven.shared.artifact.resolve.ArtifactResult> artResults =
-                       new ArrayList<org.apache.maven.shared.artifact.resolve.ArtifactResult>( artifactResults.size() );
+                        new ArrayList<org.apache.maven.shared.artifact.resolve.ArtifactResult>( artifactResults.size() );
+                    // CHECKSTYLE_ON: LineLength
 
                     for ( ArtifactResult artifactResult : artifactResults )
                     {
@@ -275,8 +274,8 @@ public class Maven31ArtifactResolver
     /**
      * Based on RepositoryUtils#toDependency(org.apache.maven.model.Dependency, ArtifactTypeRegistry)
      * 
-     * @param coordinate
-     * @param stereotypes
+     * @param coordinate {@link ArtifactCoordinate}
+     * @param stereotypes {@link ArtifactTypeRegistry
      * @return as Aether Dependency
      */
     private static Dependency toDependency( ArtifactCoordinate coordinate, ArtifactTypeRegistry stereotypes )

Modified: maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResult.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResult.java?rev=1720902&r1=1720901&r2=1720902&view=diff
==============================================================================
--- maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResult.java (original)
+++ maven/shared/trunk/maven-artifact-transfer/src/main/java/org/apache/maven/shared/artifact/resolve/internal/Maven31ArtifactResult.java Sat Dec 19 11:42:41 2015
@@ -30,15 +30,19 @@ import org.eclipse.aether.artifact.Artif
  * @author Robert Scholte
  * @since 3.0
  */
-public class Maven31ArtifactResult implements org.apache.maven.shared.artifact.resolve.ArtifactResult
+public class Maven31ArtifactResult
+    implements org.apache.maven.shared.artifact.resolve.ArtifactResult
 {
     private final ArtifactResult artifactResult;
 
+    /**
+     * @param artifactResult {@link ArtifactResult}
+     */
     public Maven31ArtifactResult( ArtifactResult artifactResult )
     {
         this.artifactResult = artifactResult;
     }
-    
+
     @Override
     public org.apache.maven.artifact.Artifact getArtifact()
     {