You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2021/10/14 16:57:48 UTC

[maven-linkcheck-plugin] 01/01: Fix JavaDoc

This is an automated email from the ASF dual-hosted git repository.

rfscholte pushed a commit to branch stabilize
in repository https://gitbox.apache.org/repos/asf/maven-linkcheck-plugin.git

commit 9966a20bca56b5c4618eac4a78b72e89416268d7
Author: rfscholte <rf...@apache.org>
AuthorDate: Thu Oct 14 18:57:35 2021 +0200

    Fix JavaDoc
---
 .../java/org/apache/maven/plugins/linkcheck/LinkcheckReport.java  | 8 ++++----
 src/main/java/org/apache/maven/plugins/linkcheck/SiteInvoker.java | 7 ++++---
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/linkcheck/LinkcheckReport.java b/src/main/java/org/apache/maven/plugins/linkcheck/LinkcheckReport.java
index e758e0f..3b2b6ff 100644
--- a/src/main/java/org/apache/maven/plugins/linkcheck/LinkcheckReport.java
+++ b/src/main/java/org/apache/maven/plugins/linkcheck/LinkcheckReport.java
@@ -345,7 +345,7 @@ public class LinkcheckReport
             basedir = new File( linkcheckOutput.getParentFile(), "tmpsite" );
             basedir.mkdirs();
 
-            List documents = null;
+            List<File> documents = null;
             try
             {
                 documents = FileUtils.getFiles( basedir, "**/*.html", null );
@@ -357,7 +357,7 @@ public class LinkcheckReport
             }
 
             // if the site was not already generated, invoke it
-            if ( documents == null || ( documents != null && documents.size() == 0 ) )
+            if ( documents == null || documents.size() == 0 )
             {
                 getLog().info( "Invoking the maven-site-plugin to ensure that all files are generated..." );
 
@@ -432,8 +432,8 @@ public class LinkcheckReport
      */
     private String[] getExcludedPages()
     {
-        List pagesToExclude =
-            ( excludedPages != null ? new ArrayList( Arrays.asList( excludedPages ) ) : new ArrayList() );
+        List<String> pagesToExclude =
+            ( excludedPages != null ? new ArrayList( Arrays.<String>asList( excludedPages ) ) : new ArrayList<String>() );
 
         // Exclude this report
         pagesToExclude.add( getOutputName() + ".html" );
diff --git a/src/main/java/org/apache/maven/plugins/linkcheck/SiteInvoker.java b/src/main/java/org/apache/maven/plugins/linkcheck/SiteInvoker.java
index 317469d..38eab4d 100644
--- a/src/main/java/org/apache/maven/plugins/linkcheck/SiteInvoker.java
+++ b/src/main/java/org/apache/maven/plugins/linkcheck/SiteInvoker.java
@@ -71,11 +71,12 @@ public class SiteInvoker
     }
 
     /**
-     * Invoke Maven for the <code>site</code> phase for a temporary Maven project using
+     * <p>Invoke Maven for the <code>site</code> phase for a temporary Maven project using
      * <code>tmpReportingOutputDirectory</code> as <code>${project.reporting.outputDirectory}</code>. This is a
-     * workaround to be sure that all site files have been correctly generated. <br/>
+     * workaround to be sure that all site files have been correctly generated.
+     * </p>
      * <b>Note 1</b>: the Maven Home should be defined in the <code>maven.home</code> Java system property or defined in
-     * <code>M2_HOME</code> system env variables. <b>Note 2</be>: we can't use <code>siteOutputDirectory</code> param
+     * <code>M2_HOME</code> system env variables. <b>Note 2</b>: we can't use <code>siteOutputDirectory</code> param
      * from site plugin because some plugins <code>${project.reporting.outputDirectory}</code> in their conf.
      *
      * @param project the MavenProject to invoke the site on. Not null.