The result of some practice with Checkstyle

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@355471 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 6 insertions, 5 deletions

diff --git a/src/java/org/apache/fop/apps/FOPException.java b/src/java/org/apache/fop/apps/FOPException.java
index 10fe45fbe..4f9d3a3a9 100644
--- a/src/java/org/apache/fop/apps/FOPException.java
+++ b/src/java/org/apache/fop/apps/FOPException.java
@@ -43,7 +43,8 @@ public class FOPException extends SAXException {
     /**
      * Constructs a new FOP exception with the specified detail message and location.
      * @param message the detail message
-     * @param systemId the system id of the FO document which is associated with the exception. May be null.
+     * @param systemId the system id of the FO document which is associated with the exception
+     *                 may be null.
      * @param line line number in the FO document which is associated with the exception.
      * @param column clolumn number in the line which is associated with the exception.
      */
@@ -96,7 +97,8 @@ public class FOPException extends SAXException {
 
     /**
      * Set a location associated with the exception.
-     * @param systemId the system id of the FO document which is associated with the exception. May be null.
+     * @param systemId the system id of the FO document which is associated with the exception;
+     *                 may be null.
      * @param line line number in the FO document which is associated with the exception.
      * @param column clolumn number in the line which is associated with the exception.
      */
@@ -108,6 +110,7 @@ public class FOPException extends SAXException {
 
     /**
      * Indicate whether a location was set.
+     * @return whether a location was set
      */
     public boolean isLocationSet() {
         // TODO: this is actually a dangerous assumption: A line
@@ -124,6 +127,7 @@ public class FOPException extends SAXException {
      *  SystemId:LL:CC: &the message&
      * </pre>
      * (the format used by most GNU tools)
+     * @return the detail message string of this FOP exception
      */
     public String getMessage() {
         if (isLocationSet()) {
diff --git a/src/java/org/apache/fop/apps/Fop.java b/src/java/org/apache/fop/apps/Fop.java
index e87c1aa4d..2694a7690 100644
--- a/src/java/org/apache/fop/apps/Fop.java
+++ b/src/java/org/apache/fop/apps/Fop.java
@@ -67,7 +67,6 @@ public class Fop implements Constants {
      * output format (ex. "application/pdf" for PDF).
      * @param outputFormat the MIME type of the output format to use (ex. "application/pdf").
      * @param ua FOUserAgent object
-     * @throws IllegalArgumentException if an unsupported renderer type was requested.
      */
     public Fop(String outputFormat, FOUserAgent ua) {
         this.outputFormat = outputFormat;
@@ -82,7 +81,6 @@ public class Fop implements Constants {
      * Constructor for FOP with a default FOUserAgent. It uses MIME types to select the 
      * output format (ex. "application/pdf" for PDF).
      * @param outputFormat the MIME type of the output format to use (ex. "application/pdf").
-     * @throws IllegalArgumentException if an unsupported renderer type was requested.
      */
     public Fop(String outputFormat) {
         this(outputFormat, null);
@@ -106,7 +104,6 @@ public class Fop implements Constants {
      * <li>Fop.RENDER_PNG</li>
      * </ul>
      * @param ua FOUserAgent object
-     * @throws IllegalArgumentException if an unsupported renderer type was requested.
      * @deprecated Use {@link org.apache.fop.apps.Fop#Fop(java.lang.String, FOUserAgent)} instead!
      *             This constructor will be removed.
      */