You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by vs...@apache.org on 2008/05/17 15:26:39 UTC

svn commit: r657351 - /maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java

Author: vsiveton
Date: Sat May 17 06:26:39 2008
New Revision: 657351

URL: http://svn.apache.org/viewvc?rev=657351&view=rev
Log:
o formatting due to r649443 

Modified:
    maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java

Modified: maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java?rev=657351&r1=657350&r2=657351&view=diff
==============================================================================
--- maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java (original)
+++ maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java Sat May 17 06:26:39 2008
@@ -271,7 +271,7 @@
     /**
      * Sets the absolute path of the Javadoc Tool executable to use. Since version 2.5, a mere directory specification
      * is sufficient to have the plugin use "javadoc" or "javadoc.exe" respectively from this directory.
-     * 
+     *
      * @since 2.3
      * @parameter expression="${javadocExecutable}"
      */
@@ -1931,25 +1931,28 @@
         return StringUtils.join( classpathElements.iterator(), File.pathSeparator );
     }
 
-    
     //TODO remove the part with ToolchainManager lookup once we depend on
     //3.0.9 (have it as prerequisite). Define as regular component field then.
-    private Toolchain getToolchain() 
+    private Toolchain getToolchain()
     {
         Toolchain tc = null;
-        try 
+        try
         {
-            if (session != null) //session is null in tests..
+            if ( session != null ) //session is null in tests..
             {
-                ToolchainManager toolchainManager = (ToolchainManager) session.getContainer().lookup(ToolchainManager.ROLE);
-                if (toolchainManager != null) 
+                ToolchainManager toolchainManager = (ToolchainManager) session.getContainer()
+                    .lookup( ToolchainManager.ROLE );
+                if ( toolchainManager != null )
                 {
-                    tc = toolchainManager.getToolchainFromBuildContext("jdk", session);
+                    tc = toolchainManager.getToolchainFromBuildContext( "jdk", session );
                 }
             }
-        } catch (ComponentLookupException componentLookupException) {
+        }
+        catch ( ComponentLookupException componentLookupException )
+        {
             //just ignore, could happen in pre-3.0.9 builds..
         }
+
         return tc;
     }
 
@@ -2374,20 +2377,20 @@
         throws IOException
     {
         Toolchain tc = getToolchain();
-        
+
         if ( tc != null )
         {
             getLog().info( "Toolchain in javadoc-plugin: " + tc );
-            if ( javadocExecutable  != null) 
-            { 
+            if ( javadocExecutable != null )
+            {
                 getLog().warn( "Toolchains are ignored, 'javadocExecutable' parameter is set to " + javadocExecutable );
-            } 
-            else 
+            }
+            else
             {
-                javadocExecutable = tc.findTool( "javadoc" ); //NOI18N
+                javadocExecutable = tc.findTool( "javadoc" );
             }
         }
-        
+
         String javadocCommand = "javadoc" + ( SystemUtils.IS_OS_WINDOWS ? ".exe" : "" );
 
         File javadocExe;