Reformatted and removed checkstyle reported warnings.

git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1726541 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/maven/plugins/acr/AcrMojo.java b/src/main/java/org/apache/maven/plugins/acr/AcrMojo.java
index 1722247..c997d83 100644
--- a/src/main/java/org/apache/maven/plugins/acr/AcrMojo.java
+++ b/src/main/java/org/apache/maven/plugins/acr/AcrMojo.java
@@ -77,7 +77,7 @@
     /**
      * Directory that resources are copied to during the build.<br/>
      * Starting with <b>3.0.0</b> the property has been renamed from <code>outputDirectory</code> to
-     * <code>maven.acr.outputDirectory</code>. 
+     * <code>maven.acr.outputDirectory</code>.
      */
     @Parameter( property = "maven.acr.outputDirectory", defaultValue = "${project.build.outputDirectory}" )
     private File outputDirectory;
@@ -126,7 +126,7 @@
     /**
      * To escape interpolated value with windows path. c:\foo\bar will be replaced with c:\\foo\\bar.<br/>
      * Starting with <b>3.0.0</b> the property has been renamed from <code>acr.escapeBackslashesInFilePath</code> to
-     * <code>maven.acr.escapeBackslashesInFilePath</code>. 
+     * <code>maven.acr.escapeBackslashesInFilePath</code>.
      */
     @Parameter( property = "maven.acr.escapeBackslashesInFilePath", defaultValue = "false" )
     private boolean escapeBackslashesInFilePath;
@@ -134,15 +134,14 @@
     /**
      * An expression preceded with this String won't be interpolated. \${foo} will be replaced with ${foo}.<br/>
      * Starting with <b>3.0.0</b> the property has been renamed from <code>acr.escapeString</code> to
-     * <code>maven.acr.escapeString</code>. 
+     * <code>maven.acr.escapeString</code>.
      */
     @Parameter( property = "maven.acr.escapeString" )
     protected String escapeString;
 
     /**
-     * To filter the deployment descriptor.
-     * Starting with <b>3.0.0</b> the property has been renamed from <code>acr.filterDeploymentDescriptor</code> to
-     * <code>maven.acr.filterDeploymentDescriptor</code>. 
+     * To filter the deployment descriptor. Starting with <b>3.0.0</b> the property has been renamed from
+     * <code>acr.filterDeploymentDescriptor</code> to <code>maven.acr.filterDeploymentDescriptor</code>.
      */
     @Parameter( property = "maven.acr.filterDeploymentDescriptor", defaultValue = "false" )
     private boolean filterDeploymentDescriptor;
@@ -163,19 +162,16 @@
     @Parameter( defaultValue = "${session}", readonly = true, required = true )
     private MavenSession session;
 
-    /**
-     * Generates the application client jar file.
-     */
+    /** {@inheritDoc} */
     public void execute()
         throws MojoExecutionException
     {
-        //todo: Add license files in META-INF directory
-
         if ( getLog().isInfoEnabled() )
         {
             getLog().info( "Building JavaEE Application client: " + jarName );
         }
 
+
         File jarFile = getAppClientJarFile( basedir, jarName );
 
         MavenArchiver archiver = new MavenArchiver();
@@ -184,8 +180,6 @@
 
         archiver.setOutputFile( jarFile );
 
-        //TODO: outputDirectory needed to be checked if it exists otherwise it is needed to be created.
-        File deploymentDescriptor = new File( outputDirectory, APP_CLIENT_XML );
         try
         {
             String[] mainJarExcludes = DEFAULT_EXCLUDES;
@@ -207,6 +201,8 @@
                 // CHECKSTYLE_ON: LineLength
             }
 
+            File deploymentDescriptor = new File( outputDirectory, APP_CLIENT_XML );
+
             if ( deploymentDescriptor.exists() )
             {
                 if ( filterDeploymentDescriptor )
@@ -240,21 +236,18 @@
         }
         catch ( ManifestException e )
         {
-            throw new MojoExecutionException(
-                                              "There was a problem reading / creating the manifest for the JavaEE Application Client  archive: "
-                                                  + e.getMessage(), e );
+            throw new MojoExecutionException( "There was a problem reading / creating the manifest for the JavaEE Application Client  archive: "
+                + e.getMessage(), e );
         }
         catch ( IOException e )
         {
-            throw new MojoExecutionException(
-                                              "There was a I/O problem creating the JavaEE Application Client archive: "
-                                                  + e.getMessage(), e );
+            throw new MojoExecutionException( "There was a I/O problem creating the JavaEE Application Client archive: "
+                + e.getMessage(), e );
         }
         catch ( DependencyResolutionRequiredException e )
         {
-            throw new MojoExecutionException(
-                                              "There was a problem resolving dependencies while creating the JavaEE Application Client archive: "
-                                                  + e.getMessage(), e );
+            throw new MojoExecutionException( "There was a problem resolving dependencies while creating the JavaEE Application Client archive: "
+                + e.getMessage(), e );
         }
         catch ( MavenFilteringException e )
         {