Fixed indentation

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

src/java/org/apache/fop/layoutmgr/BalancingColumnBreakingAlgorithm.java

@@ -55,8 +55,8 @@ public class BalancingColumnBreakingAlgorithm extends PageBreakingAlgorithm {
      * @param columnCount number of columns
      * @see PageBreakingAlgorithm
      */
-    public BalancingColumnBreakingAlgorithm                     // CSOK: ParameterNumber
-        (LayoutManager topLevelLM,
+    public BalancingColumnBreakingAlgorithm(                     // CSOK: ParameterNumber
+            LayoutManager topLevelLM,
             PageProvider pageProvider,
             PageBreakingLayoutListener layoutListener,
             int alignment, int alignmentLast,

src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java

@@ -285,11 +285,11 @@ public abstract class BreakingAlgorithm {
          * @param totalDemerits a real number
          * @param previous a node
          */
-        public KnuthNode                                        // CSOK: ParameterNumber
-            (int position, int line, int fitness,
-                         int totalWidth, int totalStretch, int totalShrink,
-                         double adjustRatio, int availableShrink, int availableStretch,
-                         int difference, double totalDemerits, KnuthNode previous) {
+        public KnuthNode(                                        // CSOK: ParameterNumber
+                int position, int line, int fitness,
+                int totalWidth, int totalStretch, int totalShrink,
+                double adjustRatio, int availableShrink, int availableStretch,
+                int difference, double totalDemerits, KnuthNode previous) {
             this.position = position;
             this.line = line;
             this.fitness = fitness;
@@ -671,11 +671,11 @@ public abstract class BreakingAlgorithm {
      * @param previous active node for the preceding breakpoint
      * @return a new node
      */
-    protected KnuthNode createNode                              // CSOK: ParameterNumber
-        (int position, int line, int fitness,
-                                   int totalWidth, int totalStretch, int totalShrink,
-                                   double adjustRatio, int availableShrink, int availableStretch,
-                                   int difference, double totalDemerits, KnuthNode previous) {
+    protected KnuthNode createNode(                              // CSOK: ParameterNumber
+            int position, int line, int fitness,
+            int totalWidth, int totalStretch, int totalShrink,
+            double adjustRatio, int availableShrink, int availableStretch,
+            int difference, double totalDemerits, KnuthNode previous) {
         return new KnuthNode(position, line, fitness,
                              totalWidth, totalStretch, totalShrink,
                              adjustRatio, availableShrink, availableStretch,

src/java/org/apache/fop/layoutmgr/inline/LineLayoutManager.java

@@ -132,10 +132,10 @@ public class LineLayoutManager extends InlineStackingLayoutManager
         private int spaceAfter;
         private int baseline;
 
-        LineBreakPosition                                       // CSOK: ParameterNumber
-            (LayoutManager lm, int index, int startIndex, int breakIndex,
-             int shrink, int stretch, int diff, double ipdA, double adjust, int ind,
-             int lh, int lw, int sb, int sa, int bl) {
+        LineBreakPosition(                                       // CSOK: ParameterNumber
+                LayoutManager lm, int index, int startIndex, int breakIndex,
+                int shrink, int stretch, int diff, double ipdA, double adjust, int ind,
+                int lh, int lw, int sb, int sa, int bl) {
             super(lm, breakIndex);
             availableShrink = shrink;
             availableStretch = stretch;
@@ -331,9 +331,9 @@ public class LineLayoutManager extends InlineStackingLayoutManager
         private int follow;
         private static final double MAX_DEMERITS = 10e6;
 
-        public LineBreakingAlgorithm                            // CSOK: ParameterNumber
-            (int pageAlign, int textAlign, int textAlignLast, int indent, int fillerWidth,
-             int lh, int ld, int fl, boolean first, int maxFlagCount, LineLayoutManager llm) {
+        public LineBreakingAlgorithm(                            // CSOK: ParameterNumber
+                int pageAlign, int textAlign, int textAlignLast, int indent, int fillerWidth,
+                int lh, int ld, int fl, boolean first, int maxFlagCount, LineLayoutManager llm) {
             super(textAlign, textAlignLast, first, false, maxFlagCount);
             pageAlignment = pageAlign;
             textIndent = indent;
@@ -410,9 +410,9 @@ public class LineLayoutManager extends InlineStackingLayoutManager
             activePossibility = -1;
         }
 
-        private LineBreakPosition makeLineBreakPosition         // CSOK: ParameterNumber
-            (KnuthSequence par, int firstElementIndex, int lastElementIndex, int availableShrink,
-             int availableStretch, int difference, double ratio, int indent) {
+        private LineBreakPosition makeLineBreakPosition(         // CSOK: ParameterNumber
+                KnuthSequence par, int firstElementIndex, int lastElementIndex, int availableShrink,
+                int availableStretch, int difference, double ratio, int indent) {
             // line height calculation - spaceBefore may differ from spaceAfter
             // by 1mpt due to rounding
             int spaceBefore = (lineHeight - lead - follow) / 2;

src/java/org/apache/fop/layoutmgr/inline/TextLayoutManager.java

@@ -81,10 +81,10 @@ public class TextLayoutManager extends LeafNodeLayoutManager {
         private boolean breakOppAfter;
         private final Font font;
 
-        AreaInfo                                                // CSOK: ParameterNumber
-            (int startIndex, int breakIndex, int wordSpaceCount, int letterSpaceCount,
-             MinOptMax areaIPD, boolean isHyphenated, boolean isSpace, boolean breakOppAfter,
-             Font font) {
+        AreaInfo(                                                // CSOK: ParameterNumber
+                int startIndex, int breakIndex, int wordSpaceCount, int letterSpaceCount,
+                MinOptMax areaIPD, boolean isHyphenated, boolean isSpace, boolean breakOppAfter,
+                Font font) {
             assert startIndex <= breakIndex;
             this.startIndex = startIndex;
             this.breakIndex = breakIndex;

src/java/org/apache/fop/layoutmgr/table/CellPart.java

@@ -61,8 +61,8 @@ class CellPart {
      * @param bpAfterLast width of (possibly optional) border- and padding-after if this
      * part will be the last one on the page
      */
-    protected CellPart                                          // CSOK: ParameterNumber
-        (PrimaryGridUnit pgu, int start, int end, boolean last,
+    protected CellPart(                                          // CSOK: ParameterNumber
+            PrimaryGridUnit pgu, int start, int end, boolean last,
             int condBeforeContentLength, int length, int condAfterContentLength,
             int bpBeforeNormal, int bpBeforeFirst,
             int bpAfterNormal, int bpAfterLast) {

src/java/org/apache/fop/layoutmgr/table/TableCellLayoutManager.java

@@ -277,10 +277,10 @@ public class TableCellLayoutManager extends BlockStackingLayoutManager
      * this row is placed on a previous page). Used to calculate the placement of the
      * row's background image if any
      */
-    public void addAreas                                        // CSOK: ParameterNumber
-        (PositionIterator parentIter, LayoutContext layoutContext, int[] spannedGridRowHeights,
-         int startRow, int endRow, int borderBeforeWhich, int borderAfterWhich, boolean firstOnPage,
-         boolean lastOnPage, RowPainter painter, int firstRowHeight) {
+    public void addAreas(                                        // CSOK: ParameterNumber
+            PositionIterator parentIter, LayoutContext layoutContext, int[] spannedGridRowHeights,
+            int startRow, int endRow, int borderBeforeWhich, int borderAfterWhich,
+            boolean firstOnPage, boolean lastOnPage, RowPainter painter, int firstRowHeight) {
         getParentArea(null);
 
         addId();

src/java/org/apache/fop/pdf/PDFCIDFont.java

@@ -56,10 +56,10 @@ public class PDFCIDFont extends PDFObject {
      * @param supplement Supplement number
      * @param descriptor CID font descriptor
      */
-    public PDFCIDFont                                           // CSOK: ParameterNumber
-        (String basefont, CIDFontType cidtype, int dw,
-         int[] w, String registry, String ordering,
-         int supplement, PDFCIDFontDescriptor descriptor) {
+    public PDFCIDFont(                                           // CSOK: ParameterNumber
+            String basefont, CIDFontType cidtype, int dw,
+            int[] w, String registry, String ordering,
+            int supplement, PDFCIDFontDescriptor descriptor) {
 
         this(basefont, cidtype, dw,
                 new PDFWArray(w),

src/java/org/apache/fop/pdf/PDFFactory.java

@@ -292,13 +292,13 @@ public class PDFFactory {
      * It should be 0 as this is the constructor for sampled functions.
      * @return the PDF function that was created
      */
-    public PDFFunction makeFunction                             // CSOK: ParameterNumber
-        (int theFunctionType, List theDomain,
-         List theRange, List theSize,
-         int theBitsPerSample, int theOrder,
-         List theEncode, List theDecode,
-         StringBuffer theFunctionDataStream,
-         List theFilter) {
+    public PDFFunction makeFunction(                             // CSOK: ParameterNumber
+            int theFunctionType, List theDomain,
+            List theRange, List theSize,
+            int theBitsPerSample, int theOrder,
+            List theEncode, List theDecode,
+            StringBuffer theFunctionDataStream,
+            List theFilter) {
         // Type 0 function
         PDFFunction function = new PDFFunction(theFunctionType, theDomain,
                                                theRange, theSize,
@@ -466,13 +466,13 @@ public class PDFFactory {
      * @param theFunction The PDF Function that maps an (x,y) location to a color
      * @return the PDF shading that was created
      */
-    public PDFShading makeShading                               // CSOK: ParameterNumber
-        (PDFResourceContext res, int theShadingType,
-         PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias, List theDomain,
-         List theMatrix,
-         PDFFunction theFunction) {
+    public PDFShading makeShading(                               // CSOK: ParameterNumber
+            PDFResourceContext res, int theShadingType,
+            PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias, List theDomain,
+            List theMatrix,
+            PDFFunction theFunction) {
         // make Shading of Type 1
         PDFShading shading = new PDFShading(theShadingType,
                                             theColorSpace, theBackground,
@@ -519,13 +519,13 @@ public class PDFFactory {
      * The default is [false, false]
      * @return the PDF shading that was created
      */
-    public PDFShading makeShading                               // CSOK: ParameterNumber
-        (PDFResourceContext res, int theShadingType,
-         PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias, List theCoords,
-         List theDomain, PDFFunction theFunction,
-         List theExtend) {
+    public PDFShading makeShading(                               // CSOK: ParameterNumber
+            PDFResourceContext res, int theShadingType,
+            PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias, List theCoords,
+            List theDomain, PDFFunction theFunction,
+            List theExtend) {
         // make Shading of Type 2 or 3
         PDFShading shading = new PDFShading(theShadingType,
                                             theColorSpace, theBackground,
@@ -573,15 +573,15 @@ public class PDFFactory {
      * @param theFunction the PDFFunction
      * @return the PDF shading that was created
      */
-    public PDFShading makeShading                               // CSOK: ParameterNumber
-        (PDFResourceContext res, int theShadingType,
-         PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias,
-         int theBitsPerCoordinate,
-         int theBitsPerComponent,
-         int theBitsPerFlag, List theDecode,
-         PDFFunction theFunction) {
+    public PDFShading makeShading(                               // CSOK: ParameterNumber
+            PDFResourceContext res, int theShadingType,
+            PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias,
+            int theBitsPerCoordinate,
+            int theBitsPerComponent,
+            int theBitsPerFlag, List theDecode,
+            PDFFunction theFunction) {
         // make Shading of type 4,6 or 7
         PDFShading shading = new PDFShading(theShadingType,
                                             theColorSpace, theBackground,
@@ -629,15 +629,15 @@ public class PDFFactory {
      * @param theFunction The PDFFunction that's mapped on to this shape
      * @return the PDF shading that was created
      */
-    public PDFShading makeShading                               // CSOK: ParameterNumber
-        (PDFResourceContext res, int theShadingType,
-         PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias,
-         int theBitsPerCoordinate,
-         int theBitsPerComponent, List theDecode,
-         int theVerticesPerRow,
-         PDFFunction theFunction) {
+    public PDFShading makeShading(                               // CSOK: ParameterNumber
+            PDFResourceContext res, int theShadingType,
+            PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias,
+            int theBitsPerCoordinate,
+            int theBitsPerComponent, List theDecode,
+            int theVerticesPerRow,
+            PDFFunction theFunction) {
         // make shading of Type 5
         PDFShading shading = new PDFShading(theShadingType,
                                             theColorSpace, theBackground,
@@ -680,12 +680,12 @@ public class PDFFactory {
      * @param thePatternDataStream The stream of pattern data to be tiled.
      * @return the PDF pattern that was created
      */
-    public PDFPattern makePattern                               // CSOK: ParameterNumber
-        (PDFResourceContext res, int thePatternType,    // 1
-         PDFResources theResources, int thePaintType, int theTilingType,
-         List theBBox, double theXStep,
-         double theYStep, List theMatrix,
-         List theXUID, StringBuffer thePatternDataStream) {
+    public PDFPattern makePattern(                               // CSOK: ParameterNumber
+            PDFResourceContext res, int thePatternType,    // 1
+            PDFResources theResources, int thePaintType, int theTilingType,
+            List theBBox, double theXStep,
+            double theYStep, List theMatrix,
+            List theXUID, StringBuffer thePatternDataStream) {
         // PDFResources theResources
         PDFPattern pattern = new PDFPattern(theResources, 1,
                                             thePaintType, theTilingType,

src/java/org/apache/fop/pdf/PDFFontDescriptor.java

@@ -40,11 +40,11 @@ public class PDFFontDescriptor extends PDFDictionary {
      * @param italicAngle the angle of the vertical dominant strokes
      * @param stemV the width of the dominant vertical stems of glyphs
      */
-    public PDFFontDescriptor                                    // CSOK: ParameterNumber
-        (String basefont, int ascent,
-         int descent, int capHeight, int flags,
-         PDFRectangle fontBBox, int italicAngle,
-         int stemV) {
+    public PDFFontDescriptor(                                    // CSOK: ParameterNumber
+            String basefont, int ascent,
+            int descent, int capHeight, int flags,
+            PDFRectangle fontBBox, int italicAngle,
+            int stemV) {
         super();
 
         put("Type", new PDFName("FontDescriptor"));

src/java/org/apache/fop/pdf/PDFFunction.java

@@ -211,11 +211,11 @@ public class PDFFunction extends PDFObject {
      * @param theFunctionType This is the type of function (0,2,3, or 4).
      * It should be 0 as this is the constructor for sampled functions.
      */
-    public PDFFunction                                          // CSOK: ParameterNumber
-        (int theFunctionType, List theDomain,
-         List theRange, List theSize, int theBitsPerSample,
-         int theOrder, List theEncode, List theDecode,
-         StringBuffer theFunctionDataStream, List theFilter) {
+    public PDFFunction(                                          // CSOK: ParameterNumber
+            int theFunctionType, List theDomain,
+            List theRange, List theSize, int theBitsPerSample,
+            int theOrder, List theEncode, List theDecode,
+            StringBuffer theFunctionDataStream, List theFilter) {
         super();
 
         this.functionType = 0;      // dang well better be 0;

src/java/org/apache/fop/pdf/PDFPattern.java

@@ -117,12 +117,12 @@ public class PDFPattern extends PDFPathPaint {
      * @param theXUID Optional vector of Integers that uniquely identify the pattern
      * @param thePatternDataStream The stream of pattern data to be tiled.
      */
-    public PDFPattern                                           // CSOK: ParameterNumber
-        (PDFResources theResources, int thePatternType,    // 1
-         int thePaintType, int theTilingType, List theBBox,
-         double theXStep, double theYStep,
-         List theMatrix, List theXUID,
-         StringBuffer thePatternDataStream) {
+    public PDFPattern(                                           // CSOK: ParameterNumber
+            PDFResources theResources, int thePatternType,    // 1
+            int thePaintType, int theTilingType, List theBBox,
+            double theXStep, double theYStep,
+            List theMatrix, List theXUID,
+            StringBuffer thePatternDataStream) {
         super();
         this.resources = theResources;
         // This next parameter is implicit to all constructors, and is

src/java/org/apache/fop/pdf/PDFShading.java

@@ -164,11 +164,11 @@ public class PDFShading extends PDFObject {
      * It's optional, the default is the identity matrix
      * @param theFunction The PDF Function that maps an (x,y) location to a color
      */
-    public PDFShading                                           // CSOK: ParameterNumber
-        (int theShadingType, PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias, List theDomain,
-         List theMatrix, PDFFunction theFunction) {
+    public PDFShading(                                           // CSOK: ParameterNumber
+            int theShadingType, PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias, List theDomain,
+            List theMatrix, PDFFunction theFunction) {
         super();
         this.shadingType = theShadingType;    // 1
         this.colorSpace = theColorSpace;
@@ -203,12 +203,12 @@ public class PDFShading extends PDFObject {
      *                  and end colors past the start and end points
      * The default is [false, false]
      */
-    public PDFShading                                           // CSOK: ParameterNumber
-        (int theShadingType, PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias, List theCoords,
-         List theDomain, PDFFunction theFunction,
-         List theExtend) {
+    public PDFShading(                                           // CSOK: ParameterNumber
+            int theShadingType, PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias, List theCoords,
+            List theDomain, PDFFunction theFunction,
+            List theExtend) {
         super();
         this.shadingType = theShadingType;    // 2 or 3
         this.colorSpace = theColorSpace;
@@ -244,12 +244,12 @@ public class PDFShading extends PDFObject {
      * @param theDecode List of Doubles see PDF 1.3 spec pages 303 to 312.
      * @param theFunction the PDFFunction
      */
-    public PDFShading                                           // CSOK: ParameterNumber
-        (int theShadingType, PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias, int theBitsPerCoordinate,
-         int theBitsPerComponent, int theBitsPerFlag,
-         List theDecode, PDFFunction theFunction) {
+    public PDFShading(                                           // CSOK: ParameterNumber
+            int theShadingType, PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias, int theBitsPerCoordinate,
+            int theBitsPerComponent, int theBitsPerFlag,
+            List theDecode, PDFFunction theFunction) {
         super();
 
         this.shadingType = theShadingType;    // 4,6 or 7
@@ -284,12 +284,12 @@ public class PDFShading extends PDFObject {
      * @param theVerticesPerRow number of vertices in each "row" of the lattice.
      * @param theFunction The PDFFunction that's mapped on to this shape
      */
-    public PDFShading                                           // CSOK: ParameterNumber
-        (int theShadingType, PDFDeviceColorSpace theColorSpace,
-         List theBackground, List theBBox,
-         boolean theAntiAlias, int theBitsPerCoordinate,
-         int theBitsPerComponent, List theDecode,
-         int theVerticesPerRow, PDFFunction theFunction) {
+    public PDFShading(                                           // CSOK: ParameterNumber
+            int theShadingType, PDFDeviceColorSpace theColorSpace,
+            List theBackground, List theBBox,
+            boolean theAntiAlias, int theBitsPerCoordinate,
+            int theBitsPerComponent, List theDecode,
+            int theVerticesPerRow, PDFFunction theFunction) {
         super();
         this.shadingType = theShadingType;    // 5
         this.colorSpace = theColorSpace;

src/java/org/apache/fop/render/AbstractPathOrientedRenderer.java

@@ -172,10 +172,10 @@ public abstract class AbstractPathOrientedRenderer extends PrintRenderer {
      * @param bpsStart the border-start traits
      * @param bpsEnd the border-end traits
      */
-    protected void drawBackground                               // CSOK: ParameterNumber
-        (float startx, float starty, float width, float height, Trait.Background back,
-         BorderProps bpsBefore, BorderProps bpsAfter,
-         BorderProps bpsStart, BorderProps bpsEnd) {
+    protected void drawBackground(                               // CSOK: ParameterNumber
+            float startx, float starty, float width, float height, Trait.Background back,
+            BorderProps bpsBefore, BorderProps bpsAfter,
+            BorderProps bpsStart, BorderProps bpsEnd) {
         if (back != null) {
             endTextObject();
 
@@ -261,10 +261,10 @@ public abstract class AbstractPathOrientedRenderer extends PrintRenderer {
      * @param bpsStart the border-start traits
      * @param bpsEnd the border-end traits
      */
-    protected void drawBorders                                  // CSOK: ParameterNumber
-        (float startx, float starty, float width, float height,
-         BorderProps bpsBefore, BorderProps bpsAfter,
-         BorderProps bpsStart, BorderProps bpsEnd) {
+    protected void drawBorders(                                  // CSOK: ParameterNumber
+            float startx, float starty, float width, float height,
+            BorderProps bpsBefore, BorderProps bpsAfter,
+            BorderProps bpsStart, BorderProps bpsEnd) {
         Rectangle2D.Float borderRect = new Rectangle2D.Float(startx, starty, width, height);
         drawBorders(borderRect, bpsBefore, bpsAfter, bpsStart, bpsEnd);
     }
@@ -282,8 +282,8 @@ public abstract class AbstractPathOrientedRenderer extends PrintRenderer {
      * @param bpsStart the border specification on the start side
      * @param bpsEnd the border specification on the end side
      */
-    protected void drawBorders                                  // CSOK: MethodLength
-        (Rectangle2D.Float borderRect,
+    protected void drawBorders(                                  // CSOK: MethodLength
+            Rectangle2D.Float borderRect,
             BorderProps bpsBefore, BorderProps bpsAfter, BorderProps bpsStart, BorderProps bpsEnd) {
         //TODO generalize each of the four conditions into using a parameterized drawBorder()
         boolean[] border = new boolean[] {
@@ -858,9 +858,9 @@ public abstract class AbstractPathOrientedRenderer extends PrintRenderer {
      * @param style the border style (one of Constants.EN_DASHED etc.)
      * @param col the color for the border segment
      */
-    protected abstract void drawBorderLine                      // CSOK: ParameterNumber
-        (float x1, float y1, float x2, float y2, boolean horz,
-         boolean startOrBefore, int style, Color col);
+    protected abstract void drawBorderLine(                      // CSOK: ParameterNumber
+            float x1, float y1, float x2, float y2, boolean horz,
+            boolean startOrBefore, int style, Color col);
 
     /** {@inheritDoc} */
     public void renderForeignObject(ForeignObject fo, Rectangle2D pos) {

src/java/org/apache/fop/render/afp/AFPPainter.java

@@ -281,8 +281,8 @@ public class AFPPainter extends AbstractIFPainter {
             return mpt / 1000f;
         }
 
-        protected void drawBorderLine                           // CSOK: ParameterNumber
-            (int x1, int y1, int x2, int y2, boolean horz,
+        protected void drawBorderLine(                           // CSOK: ParameterNumber
+                int x1, int y1, int x2, int y2, boolean horz,
                 boolean startOrBefore, int style, Color color) throws IOException {
             BorderPaintingInfo borderPaintInfo = new BorderPaintingInfo(
                     toPoints(x1), toPoints(y1), toPoints(x2), toPoints(y2),
@@ -317,9 +317,9 @@ public class AFPPainter extends AbstractIFPainter {
     }
 
     /** {@inheritDoc} */
-    public void drawText                                        // CSOK: MethodLength
-        (int x, int y, final int letterSpacing, final int wordSpacing, final int[] dx,
-         final String text) throws IFException {
+    public void drawText(                                        // CSOK: MethodLength
+            int x, int y, final int letterSpacing, final int wordSpacing, final int[] dx,
+            final String text) throws IFException {
         final int fontSize = this.state.getFontSize();
         getPaintingState().setFontSize(fontSize);
 

src/java/org/apache/fop/render/intermediate/BorderPainter.java

@@ -217,9 +217,9 @@ public abstract class BorderPainter {
      * @param color the border color
      * @throws IOException if an I/O error occurs
      */
-    protected abstract void drawBorderLine                      // CSOK: ParameterNumber
-        (int x1, int y1, int x2, int y2,
-         boolean horz, boolean startOrBefore, int style, Color color) throws IOException;
+    protected abstract void drawBorderLine(                      // CSOK: ParameterNumber
+            int x1, int y1, int x2, int y2,
+            boolean horz, boolean startOrBefore, int style, Color color) throws IOException;
 
     /**
      * Draws a line/rule.

src/java/org/apache/fop/render/intermediate/IFRenderer.java

@@ -1238,8 +1238,8 @@ public class IFRenderer extends AbstractPathOrientedRenderer {
     }
 
     /** {@inheritDoc} */
-    protected void drawBorders                                  // CSOK: ParameterNumber
-        (float startx, float starty,
+    protected void drawBorders(                                  // CSOK: ParameterNumber
+            float startx, float starty,
             float width, float height,
             BorderProps bpsBefore, BorderProps bpsAfter,
             BorderProps bpsStart, BorderProps bpsEnd) {
@@ -1252,8 +1252,8 @@ public class IFRenderer extends AbstractPathOrientedRenderer {
     }
 
     /** {@inheritDoc} */
-    protected void drawBorderLine                               // CSOK: ParameterNumber
-        (float x1, float y1, float x2, float y2, boolean horz,
+    protected void drawBorderLine(                               // CSOK: ParameterNumber
+            float x1, float y1, float x2, float y2, boolean horz,
             boolean startOrBefore, int style, Color col) {
         //Simplified implementation that is only used by renderTextDecoration()
         //drawBorders() is overridden and uses the Painter's high-level method drawBorderRect()

src/java/org/apache/fop/render/java2d/Java2DBorderPainter.java

@@ -64,8 +64,8 @@ public class Java2DBorderPainter extends BorderPainter {
     }
 
     /** {@inheritDoc} */
-    protected void drawBorderLine                               // CSOK: ParameterNumber
-        (int x1, int y1, int x2, int y2, boolean horz,
+    protected void drawBorderLine(                               // CSOK: ParameterNumber
+            int x1, int y1, int x2, int y2, boolean horz,
             boolean startOrBefore, int style, Color color) {
         float w = x2 - x1;
         float h = y2 - y1;

src/java/org/apache/fop/render/java2d/Java2DRenderer.java

@@ -536,8 +536,8 @@ public abstract class Java2DRenderer extends AbstractPathOrientedRenderer implem
     }
 
     /** {@inheritDoc} */
-    protected void drawBorderLine                               // CSOK: ParameterNumber
-        (float x1, float y1, float x2, float y2,
+    protected void drawBorderLine(                               // CSOK: ParameterNumber
+            float x1, float y1, float x2, float y2,
             boolean horz, boolean startOrBefore, int style, Color col) {
         Graphics2D g2d = state.getGraph();
         float width = x2 - x1;

src/java/org/apache/fop/render/pcl/PCLPainter.java

@@ -390,11 +390,11 @@ public class PCLPainter extends AbstractIFPainter implements PCLConstants {
 
     private static final double SAFETY_MARGIN_FACTOR = 0.05;
 
-    private Rectangle getTextBoundingBox                        // CSOK: ParameterNumber
-        (int x, int y,
-         int letterSpacing, int wordSpacing, int[] dx,
-         String text,
-         Font font, FontMetricsMapper metrics) {
+    private Rectangle getTextBoundingBox(                        // CSOK: ParameterNumber
+            int x, int y,
+            int letterSpacing, int wordSpacing, int[] dx,
+            String text,
+            Font font, FontMetricsMapper metrics) {
         int maxAscent = metrics.getMaxAscent(font.getFontSize()) / 1000;
         int descent = metrics.getDescender(font.getFontSize()) / 1000; //is negative
         int safetyMargin = (int)(SAFETY_MARGIN_FACTOR * font.getFontSize());

src/java/org/apache/fop/render/pdf/PDFBorderPainter.java

@@ -50,9 +50,9 @@ public class PDFBorderPainter extends BorderPainter {
     }
 
     /** {@inheritDoc} */
-    protected void drawBorderLine                               // CSOK: ParameterNumber
-        (int x1, int y1, int x2, int y2, boolean horz,
-         boolean startOrBefore, int style, Color col) {
+    protected void drawBorderLine(                               // CSOK: ParameterNumber
+            int x1, int y1, int x2, int y2, boolean horz,
+            boolean startOrBefore, int style, Color col) {
        drawBorderLine(generator, x1 / 1000f, y1 / 1000f, x2 / 1000f, y2 / 1000f,
                horz, startOrBefore, style, col);
     }
@@ -61,10 +61,10 @@ public class PDFBorderPainter extends BorderPainter {
      * @param generator pdf content generator
      * @see BorderPainter#drawBorderLine
      */
-    public static void drawBorderLine       // CSOK: ParameterNumber|MethodLength
-        (PDFContentGenerator generator,
-         float x1, float y1, float x2, float y2, boolean horz,  // CSOK: JavadocMethod
-         boolean startOrBefore, int style, Color col) {         // CSOK: JavadocMethod
+    public static void drawBorderLine(       // CSOK: ParameterNumber|MethodLength
+            PDFContentGenerator generator,
+            float x1, float y1, float x2, float y2, boolean horz,  // CSOK: JavadocMethod
+            boolean startOrBefore, int style, Color col) {         // CSOK: JavadocMethod
         float colFactor;
         float w = x2 - x1;
         float h = y2 - y1;

src/java/org/apache/fop/render/ps/PSBorderPainter.java

@@ -52,9 +52,9 @@ public class PSBorderPainter extends BorderPainter {
     }
 
     /** {@inheritDoc} */
-    protected void drawBorderLine                               // CSOK: ParameterNumber
-        (int x1, int y1, int x2, int y2, boolean horz,
-         boolean startOrBefore, int style, Color col) throws IOException {
+    protected void drawBorderLine(                               // CSOK: ParameterNumber
+            int x1, int y1, int x2, int y2, boolean horz,
+            boolean startOrBefore, int style, Color col) throws IOException {
        drawBorderLine(generator, toPoints(x1), toPoints(y1), toPoints(x2), toPoints(y2),
                horz, startOrBefore, style, col);
     }
@@ -71,10 +71,10 @@ public class PSBorderPainter extends BorderPainter {
      * @param gen ps content generator
      * @see BorderPainter#drawBorderLine
      */
-    public static void drawBorderLine                           // CSOK: ParameterNumber
-        (PSGenerator gen,
-         float x1, float y1, float x2, float y2, boolean horz,  // CSOK: JavadocMethod
-         boolean startOrBefore, int style, Color col)           // CSOK: JavadocMethod
+    public static void drawBorderLine(                           // CSOK: ParameterNumber
+            PSGenerator gen,
+            float x1, float y1, float x2, float y2, boolean horz,  // CSOK: JavadocMethod
+            boolean startOrBefore, int style, Color col)           // CSOK: JavadocMethod
         throws IOException {                                    // CSOK: JavadocMethod
         float w = x2 - x1;
         float h = y2 - y1;

src/java/org/apache/fop/render/ps/PSPainter.java

@@ -403,10 +403,10 @@ public class PSPainter extends AbstractIFPainter {
         }
     }
 
-    private void writeText                                      // CSOK: ParameterNumber
-        (String text, int start, int len,
-         int letterSpacing, int wordSpacing, int[] dx,
-         Font font, Typeface tf) throws IOException {
+    private void writeText(                                      // CSOK: ParameterNumber
+            String text, int start, int len,
+            int letterSpacing, int wordSpacing, int[] dx,
+            Font font, Typeface tf) throws IOException {
         PSGenerator generator = getGenerator();
         int end = start + len;
         int initialSize = len;

src/java/org/apache/fop/render/txt/TXTRenderer.java

@@ -494,9 +494,9 @@ public class TXTRenderer extends AbstractPathOrientedRenderer {
     /**
      * {@inheritDoc}
      */
-    protected void drawBorderLine                               // CSOK: ParameterNumber
-        (float x1, float y1, float x2, float y2,
-         boolean horz, boolean startOrBefore, int style, Color col) {
+    protected void drawBorderLine(                               // CSOK: ParameterNumber
+            float x1, float y1, float x2, float y2,
+            boolean horz, boolean startOrBefore, int style, Color col) {
 
         int borderHeight = bm.getHeight();
         int borderWidth = bm.getWidth();

src/java/org/apache/fop/util/ColorExt.java

@@ -118,10 +118,10 @@ public final class ColorExt extends Color {
      *            ICC color values
      * @return the requested color object
      */
-    public static ColorExt createFromSvgIccColor                // CSOK: ParameterNumber
-        (float red, float green,
-         float blue, float opacity, String profileName, String profileHref,
-         ColorSpace profileCS, float[] colorValues) {
+    public static ColorExt createFromSvgIccColor(                // CSOK: ParameterNumber
+            float red, float green,
+            float blue, float opacity, String profileName, String profileHref,
+            ColorSpace profileCS, float[] colorValues) {
         //TODO this method is not referenced by FOP, can it be deleted?
         ColorExt ce = new ColorExt(red, green, blue, opacity);
         ce.rgbReplacementRed = -1;