From 80a0e76ff22136ece626e5516aeeabd6ef761b3b Mon Sep 17 00:00:00 2001
From: Jaikiran Pai <jaikiran@apache.org>
Date: Fri, 17 Aug 2018 12:52:09 +0530
Subject: [PATCH] Fix javadoc warnings in junitlauncher task

---
 .../junitlauncher/JUnitLauncherTask.java       |  3 +--
 .../junitlauncher/LaunchDefinition.java        | 18 ++++++------------
 .../junitlauncher/StandaloneLauncher.java      |  2 +-
 3 files changed, 8 insertions(+), 15 deletions(-)

diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java
index a328e4b1a..028397e87 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java
@@ -62,8 +62,7 @@ import static org.apache.tools.ant.taskdefs.optional.junitlauncher.Constants.LD_
  * platform for test execution. It however, is solely the JUnit 5 platform, backed by test engines that
  * decide and execute the tests.
  *
- * @see <a href="https://junit.org/junit5/">JUnit 5 documentation</a> for more details
- * on how JUnit manages the platform and the test engines.
+ * @see <a href="https://junit.org/junit5/">JUnit 5 documentation</a>
  */
 public class JUnitLauncherTask extends Task {
 
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LaunchDefinition.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LaunchDefinition.java
index d3e5ae36c..3dd5350f0 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LaunchDefinition.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LaunchDefinition.java
@@ -27,49 +27,43 @@ import java.util.List;
 public interface LaunchDefinition {
 
     /**
-     * Returns the {@link TestDefinition tests} that have to be launched
+     * @return Returns the {@link TestDefinition tests} that have to be launched
      *
-     * @return
      */
     List<TestDefinition> getTests();
 
     /**
-     * Returns the default {@link ListenerDefinition listeners} that will be used
+     * @return Returns the default {@link ListenerDefinition listeners} that will be used
      * for the tests, if the {@link #getTests() tests} themselves don't specify any
      *
-     * @return
      */
     List<ListenerDefinition> getListeners();
 
     /**
-     * Returns true if a summary needs to be printed out after the execution of the
+     * @return Returns true if a summary needs to be printed out after the execution of the
      * tests. False otherwise.
      *
-     * @return
      */
     boolean isPrintSummary();
 
     /**
-     * Returns true if any remaining tests launch need to be stopped if any test execution
+     * @return Returns true if any remaining tests launch need to be stopped if any test execution
      * failed. False otherwise.
      *
-     * @return
      */
     boolean isHaltOnFailure();
 
     /**
-     * Returns the {@link ClassLoader} that has to be used for launching and execution of the
+     * @return Returns the {@link ClassLoader} that has to be used for launching and execution of the
      * tests
      *
-     * @return
      */
     ClassLoader getClassLoader();
 
     /**
-     * Returns the {@link TestExecutionContext} that will be passed to {@link TestResultFormatter#setContext(TestExecutionContext)
+     * @return Returns the {@link TestExecutionContext} that will be passed to {@link TestResultFormatter#setContext(TestExecutionContext)
      * result formatters} which are applicable during the execution of the tests.
      *
-     * @return
      */
     TestExecutionContext getTestExecutionContext();
 }
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/StandaloneLauncher.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/StandaloneLauncher.java
index 7e342cc80..8f43da373 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/StandaloneLauncher.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/StandaloneLauncher.java
@@ -59,7 +59,7 @@ public class StandaloneLauncher {
      * Entry point to launching the forked test.
      *
      * @param args The arguments passed to this program for launching the tests
-     * @throws Exception
+     * @throws Exception If any exception occurs during the execution
      */
     public static void main(final String[] args) throws Exception {
         // The main responsibility of this entry point is to create a LaunchDefinition,