Amended checkstyle errors and return carriages

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

diff --git a/src/java/org/apache/fop/afp/fonts/AFPBase12FontCollection.java b/src/java/org/apache/fop/afp/fonts/AFPBase12FontCollection.java
index c872a08ccca7d3d58cf51193fc27d887c9e22915..244e6122cbce0e5c71b3df65374502aeb02b28d6 100644 (file)
--- a/src/java/org/apache/fop/afp/fonts/AFPBase12FontCollection.java
+++ b/src/java/org/apache/fop/afp/fonts/AFPBase12FontCollection.java
@@ -43,6 +43,9 @@ public class AFPBase12FontCollection implements FontCollection {
 
     private final AFPEventProducer eventProducer;
 
+    /**
+     * @param eventProducer the AFP-specific event producer
+     */
     public AFPBase12FontCollection(AFPEventProducer eventProducer) {
         this.eventProducer = eventProducer;
     }

diff --git a/src/java/org/apache/fop/afp/modca/GraphicsObject.java b/src/java/org/apache/fop/afp/modca/GraphicsObject.java
index 66876a5ac53cc41ac5129eff9c0cc92aca1c9f24..19d4618be8fe63f5b1c0e3cec2bb22ba2a9ffb8e 100644 (file)
--- a/src/java/org/apache/fop/afp/modca/GraphicsObject.java
+++ b/src/java/org/apache/fop/afp/modca/GraphicsObject.java
@@ -325,6 +325,7 @@ public class GraphicsObject extends AbstractDataObject {
      * @param str the string
      * @param x the x coordinate
      * @param y the y coordinate
+     * @param charSet the character set associated with the string
      */
     public void addString(String str, int x, int y, CharacterSet charSet) {
         addObject(new GraphicsCharacterString(str, x, y, charSet));

diff --git a/src/java/org/apache/fop/fonts/type1/Type1FontLoader.java b/src/java/org/apache/fop/fonts/type1/Type1FontLoader.java
index b5b573eca51756982956be2f27e2240c98935c1b..1e93763e86befe8d5ed817f5c98e791653fe5674 100644 (file)
--- a/src/java/org/apache/fop/fonts/type1/Type1FontLoader.java
+++ b/src/java/org/apache/fop/fonts/type1/Type1FontLoader.java
@@ -62,7 +62,7 @@ public class Type1FontLoader extends FontLoader {
         return pfbURI.substring(0, pfbURI.length() - 4) + "." + pfmExt;
     }
 
-    private static final String[] AFM_EXTENSIONS = new String[] { ".AFM", ".afm", ".Afm" };
+    private static final String[] AFM_EXTENSIONS = new String[] {".AFM", ".afm", ".Afm"};
 
     /** {@inheritDoc} */
     @Override

diff --git a/src/java/org/apache/fop/render/AbstractPathOrientedRenderer.java b/src/java/org/apache/fop/render/AbstractPathOrientedRenderer.java
index 83d3d84a2554d86174d3a377aead15cd9e1d1c1c..333248f4d2220aa33576b36a1817132509d14cb1 100644 (file)
--- a/src/java/org/apache/fop/render/AbstractPathOrientedRenderer.java
+++ b/src/java/org/apache/fop/render/AbstractPathOrientedRenderer.java
@@ -57,6 +57,9 @@ import org.apache.fop.traits.BorderProps;
  */
 public abstract class AbstractPathOrientedRenderer extends PrintRenderer {
 
+    /**
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public AbstractPathOrientedRenderer(FOUserAgent userAgent) {
         super(userAgent);
     }

diff --git a/src/java/org/apache/fop/render/PrintRenderer.java b/src/java/org/apache/fop/render/PrintRenderer.java
index 05e9cb7af6ab17a5aec35bf6e69390dd04f85769..4e49adc258261c84a23c353755b217d769ab50c6 100644 (file)
--- a/src/java/org/apache/fop/render/PrintRenderer.java
+++ b/src/java/org/apache/fop/render/PrintRenderer.java
@@ -43,6 +43,9 @@ import org.apache.fop.fonts.base14.Base14FontCollection;
 /** Abstract base class of "Print" type renderers.  */
 public abstract class PrintRenderer extends AbstractRenderer {
 
+    /**
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public PrintRenderer(FOUserAgent userAgent) {
         super(userAgent);
     }

diff --git a/src/java/org/apache/fop/render/afp/AFPRendererConfigurator.java b/src/java/org/apache/fop/render/afp/AFPRendererConfigurator.java
index bfef1b0cc29c37fbb071b708c91ee6cadd0657cb..23449ce4e8be35ecb7a2cb85340f578c496ed523 100644 (file)
--- a/src/java/org/apache/fop/render/afp/AFPRendererConfigurator.java
+++ b/src/java/org/apache/fop/render/afp/AFPRendererConfigurator.java
@@ -204,8 +204,10 @@ public class AFPRendererConfigurator extends PrintRendererConfigurator
                                 "org.apache.fop.fonts.base14." + base14).asSubclass(Typeface.class);
                         try {
                             Typeface tf = clazz.newInstance();
-                            font.addCharacterSet(sizeMpt, CharacterSetBuilder.getSingleByteInstance()
-                                      .build(characterset, codepage, encoding, tf, eventProducer));
+                            font.addCharacterSet(sizeMpt,
+                                    CharacterSetBuilder.getSingleByteInstance()
+                                                       .build(characterset, codepage, encoding, tf,
+                                                               eventProducer));
                         } catch (Exception ie) {
                             String msg = "The base 14 font class " + clazz.getName()
                             + " could not be instantiated";

diff --git a/src/java/org/apache/fop/render/bitmap/PNGRenderer.java b/src/java/org/apache/fop/render/bitmap/PNGRenderer.java
index 1c6f2e33df6ad0c3e81bd6cca4454142817daef0..6f45d18ca721dbc8af86fa8f1b963fccbea64fea 100644 (file)
--- a/src/java/org/apache/fop/render/bitmap/PNGRenderer.java
+++ b/src/java/org/apache/fop/render/bitmap/PNGRenderer.java
@@ -41,6 +41,9 @@ import org.apache.fop.render.java2d.Java2DRenderer;
  */
 public class PNGRenderer extends Java2DRenderer {
 
+    /**
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public PNGRenderer(FOUserAgent userAgent) {
         super(userAgent);
     }

diff --git a/src/java/org/apache/fop/render/bitmap/TIFFRenderer.java b/src/java/org/apache/fop/render/bitmap/TIFFRenderer.java
index 0e2364d35d36d46306ca68136a8550aa02a18ab7..3cdcf2cdcf19e815071d40994f01238a21f93632 100644 (file)
--- a/src/java/org/apache/fop/render/bitmap/TIFFRenderer.java
+++ b/src/java/org/apache/fop/render/bitmap/TIFFRenderer.java
@@ -82,7 +82,11 @@ public class TIFFRenderer extends Java2DRenderer implements TIFFConstants {
         return MIME_TYPE;
     }
 
-    /** Creates TIFF renderer. */
+    /**
+     * Creates TIFF renderer.
+     *
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public TIFFRenderer(FOUserAgent userAgent) {
         super(userAgent);
         writerParams = new ImageWriterParams();

diff --git a/src/java/org/apache/fop/render/intermediate/IFRenderer.java b/src/java/org/apache/fop/render/intermediate/IFRenderer.java
index d5d5a154a99547cac2b0643f1344b54699912ea7..9d8fb6e4bdbf0cef1eb46c7c7bae622163312902 100644 (file)
--- a/src/java/org/apache/fop/render/intermediate/IFRenderer.java
+++ b/src/java/org/apache/fop/render/intermediate/IFRenderer.java
@@ -158,6 +158,8 @@ public class IFRenderer extends AbstractPathOrientedRenderer {
 
     /**
      * Main constructor
+     *
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
      */
     public IFRenderer(FOUserAgent userAgent) {
         super(userAgent);

diff --git a/src/java/org/apache/fop/render/java2d/Java2DRenderer.java b/src/java/org/apache/fop/render/java2d/Java2DRenderer.java
index 3c7006588cadf822104805e77b23acdf22999a98..1d1697607eea5beef0a07ece50c340ca46d0a812 100644 (file)
--- a/src/java/org/apache/fop/render/java2d/Java2DRenderer.java
+++ b/src/java/org/apache/fop/render/java2d/Java2DRenderer.java
@@ -146,7 +146,11 @@ public abstract class Java2DRenderer extends AbstractPathOrientedRenderer implem
 
     private GeneralPath currentPath = null;
 
-    /** Default constructor */
+    /**
+     * Default constructor
+     *
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public Java2DRenderer(FOUserAgent userAgent) {
         super(userAgent);
 

diff --git a/src/java/org/apache/fop/render/print/PageableRenderer.java b/src/java/org/apache/fop/render/print/PageableRenderer.java
index 08d5b0f041805bef23110a9b3d2f7deb93bfc43d..70a7f84a61f59c1ae84bee4efde43bff2fb6159e 100644 (file)
--- a/src/java/org/apache/fop/render/print/PageableRenderer.java
+++ b/src/java/org/apache/fop/render/print/PageableRenderer.java
@@ -72,6 +72,8 @@ public class PageableRenderer extends Java2DRenderer implements Pageable {
 
     /**
      * Creates a new PageableRenderer.
+     *
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
      */
     public PageableRenderer(FOUserAgent userAgent) {
         super(userAgent);

diff --git a/src/java/org/apache/fop/render/print/PrintRenderer.java b/src/java/org/apache/fop/render/print/PrintRenderer.java
index ea68bc9b2aa939cbbdece771439911418443e7e4..1540acd23121d4654d2117162de513bc8ff1b10d 100644 (file)
--- a/src/java/org/apache/fop/render/print/PrintRenderer.java
+++ b/src/java/org/apache/fop/render/print/PrintRenderer.java
@@ -54,6 +54,8 @@ public class PrintRenderer extends PageableRenderer {
     /**
      * Creates a new PrintRenderer with the options set through the renderer options if a custom
      * PrinterJob is not given in FOUserAgent's renderer options.
+     *
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
      */
     public PrintRenderer(FOUserAgent userAgent) {
         super(userAgent);

diff --git a/src/java/org/apache/fop/render/txt/TXTRenderer.java b/src/java/org/apache/fop/render/txt/TXTRenderer.java
index 74c5b84a8b4547f00009c065a3a265aa19e70ce1..eecb6a623893c113d9d9952d9f6943d5498341b7 100644 (file)
--- a/src/java/org/apache/fop/render/txt/TXTRenderer.java
+++ b/src/java/org/apache/fop/render/txt/TXTRenderer.java
@@ -110,6 +110,8 @@ public class TXTRenderer extends AbstractPathOrientedRenderer {
 
     /**
      * Constructs a newly allocated <code>TXTRenderer</code> object.
+     *
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
      */
     public TXTRenderer(FOUserAgent userAgent) {
         super(userAgent);

diff --git a/src/java/org/apache/fop/render/xml/AbstractXMLRenderer.java b/src/java/org/apache/fop/render/xml/AbstractXMLRenderer.java
index 9c938199f9819602aa439d8336269953af969e1f..149111e0dce21a8756a4993d1e70e189879f56b8 100644 (file)
--- a/src/java/org/apache/fop/render/xml/AbstractXMLRenderer.java
+++ b/src/java/org/apache/fop/render/xml/AbstractXMLRenderer.java
@@ -49,6 +49,9 @@ import org.apache.fop.render.RendererContext;
 /** Abstract xml renderer base class. */
 public abstract class AbstractXMLRenderer extends PrintRenderer {
 
+    /**
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public AbstractXMLRenderer(FOUserAgent userAgent) {
         super(userAgent);
     }

diff --git a/src/java/org/apache/fop/render/xml/XMLRenderer.java b/src/java/org/apache/fop/render/xml/XMLRenderer.java
index 3ae49b1214cdf2d9261e72c0bc71292da52e92ed..d2cd55abe99705cc4e27a17f45fcbde6880ab201 100644 (file)
--- a/src/java/org/apache/fop/render/xml/XMLRenderer.java
+++ b/src/java/org/apache/fop/render/xml/XMLRenderer.java
@@ -112,6 +112,9 @@ public class XMLRenderer extends AbstractXMLRenderer {
 
     private int pageSequenceIndex;
 
+    /**
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public XMLRenderer(FOUserAgent userAgent) {
         super(userAgent);
         context = new RendererContext(this, XML_MIME_TYPE);

diff --git a/src/sandbox/org/apache/fop/render/svg/SVGRenderer.java b/src/sandbox/org/apache/fop/render/svg/SVGRenderer.java
index 246062a3285ccbab7f2f65fc2d4bfd06256b8745..0e575c736a4f3e6a109493a063ef5347494bce2f 100644 (file)
--- a/src/sandbox/org/apache/fop/render/svg/SVGRenderer.java
+++ b/src/sandbox/org/apache/fop/render/svg/SVGRenderer.java
@@ -70,6 +70,9 @@ public class SVGRenderer extends Java2DRenderer {
     /** Helper class for generating multiple files */
     private MultiFileRenderingUtil multiFileUtil;
 
+    /**
+     * @param userAgent the user agent that contains configuration details. This cannot be null.
+     */
     public SVGRenderer(FOUserAgent userAgent) {
         super(userAgent);
     }

diff --git a/test/java/org/apache/fop/fonts/truetype/TTFFontLoaderTestCase.java b/test/java/org/apache/fop/fonts/truetype/TTFFontLoaderTestCase.java
index 0d6fc1d4803f4a083a5a31cf1fa8dbe241cd484a..aadf877e046f2c37ac7517484fb1a2a5800050c5 100644 (file)
--- a/test/java/org/apache/fop/fonts/truetype/TTFFontLoaderTestCase.java
+++ b/test/java/org/apache/fop/fonts/truetype/TTFFontLoaderTestCase.java
@@ -1,57 +1,57 @@
-/*\r
- * Licensed to the Apache Software Foundation (ASF) under one or more\r
- * contributor license agreements.  See the NOTICE file distributed with\r
- * this work for additional information regarding copyright ownership.\r
- * The ASF licenses this file to You under the Apache License, Version 2.0\r
- * (the "License"); you may not use this file except in compliance with\r
- * the License.  You may obtain a copy of the License at\r
- *\r
- *      http://www.apache.org/licenses/LICENSE-2.0\r
- *\r
- * Unless required by applicable law or agreed to in writing, software\r
- * distributed under the License is distributed on an "AS IS" BASIS,\r
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
- * See the License for the specific language governing permissions and\r
- * limitations under the License.\r
- */\r
-\r
-/* $Id$ */\r
-\r
-package org.apache.fop.fonts.truetype;\r
-\r
-import static org.junit.Assert.assertFalse;\r
-import static org.junit.Assert.assertTrue;\r
-\r
-import java.io.File;\r
-import java.io.IOException;\r
-\r
-import org.junit.Test;\r
-\r
-import org.apache.fop.fonts.EncodingMode;\r
-import org.apache.fop.fonts.FontManager;\r
-import org.apache.fop.fonts.FontResolver;\r
-\r
-/**\r
- * Test case for {@link TTFFontLoader}.\r
- */\r
-public class TTFFontLoaderTestCase {\r
-\r
-    @Test\r
-    public void testUseKerning() throws IOException {\r
-        File file = new File("test/resources/fonts/ttf/DejaVuLGCSerif.ttf");\r
-        String absoluteFilePath = file.toURL().toExternalForm();\r
-        FontResolver resolver = FontManager.createMinimalFontResolver();\r
-        String fontName = "Deja Vu";\r
-        boolean embedded = false;\r
-        boolean useKerning = true;\r
-\r
-        TTFFontLoader fontLoader = new TTFFontLoader(absoluteFilePath, fontName, embedded,\r
-                EncodingMode.AUTO, useKerning, resolver);\r
-        assertTrue(fontLoader.getFont().hasKerningInfo());\r
-        useKerning = false;\r
-\r
-        fontLoader = new TTFFontLoader(absoluteFilePath, fontName, embedded, EncodingMode.AUTO,\r
-                useKerning, resolver);\r
-        assertFalse(fontLoader.getFont().hasKerningInfo());\r
-    }\r
-}\r
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.fonts.truetype;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+import java.io.File;
+import java.io.IOException;
+
+import org.junit.Test;
+
+import org.apache.fop.fonts.EncodingMode;
+import org.apache.fop.fonts.FontManager;
+import org.apache.fop.fonts.FontResolver;
+
+/**
+ * Test case for {@link TTFFontLoader}.
+ */
+public class TTFFontLoaderTestCase {
+
+    @Test
+    public void testUseKerning() throws IOException {
+        File file = new File("test/resources/fonts/ttf/DejaVuLGCSerif.ttf");
+        String absoluteFilePath = file.toURI().toURL().toExternalForm();
+        FontResolver resolver = FontManager.createMinimalFontResolver();
+        String fontName = "Deja Vu";
+        boolean embedded = false;
+        boolean useKerning = true;
+
+        TTFFontLoader fontLoader = new TTFFontLoader(absoluteFilePath, fontName, embedded,
+                EncodingMode.AUTO, useKerning, resolver);
+        assertTrue(fontLoader.getFont().hasKerningInfo());
+        useKerning = false;
+
+        fontLoader = new TTFFontLoader(absoluteFilePath, fontName, embedded, EncodingMode.AUTO,
+                useKerning, resolver);
+        assertFalse(fontLoader.getFont().hasKerningInfo());
+    }
+}

diff --git a/test/java/org/apache/fop/pdf/PDFObjectTestCase.java b/test/java/org/apache/fop/pdf/PDFObjectTestCase.java
index d4ad87d78dc4c93bdcc840b51aad2887066bc241..ee9512d88ebf44ddf768d19ea242e2adf0dfd183 100644 (file)
--- a/test/java/org/apache/fop/pdf/PDFObjectTestCase.java
+++ b/test/java/org/apache/fop/pdf/PDFObjectTestCase.java
@@ -19,13 +19,17 @@
 
 package org.apache.fop.pdf;
 
-import org.junit.Test;
-import org.junit.Before;
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 
+import org.junit.Before;
+import org.junit.Test;
+
 /**
  * Tests the PDFObject class.
  */