]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
findbugs-reported bug squashing; 959 bugs left (findbugs 1.3.9)
authorSimon Pepping <spepping@apache.org>
Wed, 17 Nov 2010 19:45:27 +0000 (19:45 +0000)
committerSimon Pepping <spepping@apache.org>
Wed, 17 Nov 2010 19:45:27 +0000 (19:45 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@1036179 13f79535-47bb-0310-9956-ffa450edef68

105 files changed:
checkstyle-5.1.xml
examples/embedding/java/embedding/ExampleFO2PDF.java
examples/embedding/java/embedding/MultipleFO2PDF.java
src/java/org/apache/fop/afp/AbstractAFPPainter.java
src/java/org/apache/fop/afp/modca/AbstractAFPObject.java
src/java/org/apache/fop/apps/PageSequenceResults.java
src/java/org/apache/fop/area/Area.java
src/java/org/apache/fop/area/AreaTreeModel.java
src/java/org/apache/fop/area/AreaTreeParser.java
src/java/org/apache/fop/area/BeforeFloat.java
src/java/org/apache/fop/area/Block.java
src/java/org/apache/fop/area/BlockParent.java
src/java/org/apache/fop/area/BlockViewport.java
src/java/org/apache/fop/area/BodyRegion.java
src/java/org/apache/fop/area/CTM.java
src/java/org/apache/fop/area/Footnote.java
src/java/org/apache/fop/area/LineArea.java
src/java/org/apache/fop/area/LinkResolver.java
src/java/org/apache/fop/area/MainReference.java
src/java/org/apache/fop/area/NormalFlow.java
src/java/org/apache/fop/area/Page.java
src/java/org/apache/fop/area/PageViewport.java
src/java/org/apache/fop/area/RegionReference.java
src/java/org/apache/fop/area/RegionViewport.java
src/java/org/apache/fop/area/Span.java
src/java/org/apache/fop/area/Trait.java
src/java/org/apache/fop/area/inline/AbstractTextArea.java
src/java/org/apache/fop/area/inline/Anchor.java
src/java/org/apache/fop/area/inline/Container.java
src/java/org/apache/fop/area/inline/FilledArea.java
src/java/org/apache/fop/area/inline/Image.java
src/java/org/apache/fop/area/inline/InlineArea.java
src/java/org/apache/fop/area/inline/InlineBlockParent.java
src/java/org/apache/fop/area/inline/InlineParent.java
src/java/org/apache/fop/area/inline/Leader.java
src/java/org/apache/fop/area/inline/Space.java
src/java/org/apache/fop/area/inline/SpaceArea.java
src/java/org/apache/fop/area/inline/TextArea.java
src/java/org/apache/fop/area/inline/UnresolvedPageNumber.java
src/java/org/apache/fop/area/inline/Viewport.java
src/java/org/apache/fop/area/inline/WordArea.java
src/java/org/apache/fop/cli/CommandLineOptions.java
src/java/org/apache/fop/datatypes/LengthBase.java
src/java/org/apache/fop/fo/FONode.java
src/java/org/apache/fop/fo/FOText.java
src/java/org/apache/fop/fo/extensions/xmp/XMPMetadata.java
src/java/org/apache/fop/fo/properties/Property.java
src/java/org/apache/fop/fonts/FontInfo.java
src/java/org/apache/fop/fonts/FontInfoConfigurator.java
src/java/org/apache/fop/fonts/FontLoader.java
src/java/org/apache/fop/fonts/apps/PFMReader.java
src/java/org/apache/fop/fonts/substitute/FontSubstitutions.java
src/java/org/apache/fop/fonts/substitute/FontWeightRange.java
src/java/org/apache/fop/fonts/type1/PFMFile.java
src/java/org/apache/fop/fonts/type1/Type1FontLoader.java
src/java/org/apache/fop/hyphenation/ByteVector.java
src/java/org/apache/fop/hyphenation/CharVector.java
src/java/org/apache/fop/hyphenation/Hyphen.java
src/java/org/apache/fop/hyphenation/Hyphenator.java
src/java/org/apache/fop/hyphenation/TernaryTree.java
src/java/org/apache/fop/layoutmgr/AbstractBreaker.java
src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java
src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java
src/java/org/apache/fop/layoutmgr/InlineKnuthSequence.java
src/java/org/apache/fop/layoutmgr/UnresolvedListElementWithLength.java
src/java/org/apache/fop/layoutmgr/inline/ImageLayout.java
src/java/org/apache/fop/layoutmgr/inline/LeafNodeLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/LineLayoutManager.java
src/java/org/apache/fop/pdf/PDFObject.java
src/java/org/apache/fop/render/AbstractConfigurator.java
src/java/org/apache/fop/render/AbstractGenericSVGHandler.java
src/java/org/apache/fop/render/AbstractRenderer.java
src/java/org/apache/fop/render/PrintRendererConfigurator.java
src/java/org/apache/fop/render/XMLHandler.java
src/java/org/apache/fop/render/XMLHandlerConfigurator.java
src/java/org/apache/fop/render/XMLHandlerRegistry.java
src/java/org/apache/fop/render/afp/AFPRendererImageInfo.java
src/java/org/apache/fop/render/afp/extensions/AFPExtensionHandler.java
src/java/org/apache/fop/render/awt/viewer/PageChangeEvent.java
src/java/org/apache/fop/render/bitmap/TIFFRenderer.java
src/java/org/apache/fop/render/intermediate/IFParser.java
src/java/org/apache/fop/render/intermediate/IFRenderer.java
src/java/org/apache/fop/render/intermediate/IFSerializer.java
src/java/org/apache/fop/render/intermediate/extensions/AbstractAction.java
src/java/org/apache/fop/render/intermediate/extensions/ActionSet.java
src/java/org/apache/fop/render/intermediate/extensions/DocumentNavigationHandler.java
src/java/org/apache/fop/render/intermediate/extensions/GoToXYAction.java
src/java/org/apache/fop/render/intermediate/extensions/URIAction.java
src/java/org/apache/fop/render/pdf/PDFDocumentNavigationHandler.java
src/java/org/apache/fop/render/pdf/extensions/PDFExtensionHandler.java
src/java/org/apache/fop/render/ps/PSFontUtils.java
src/java/org/apache/fop/render/ps/PSImageFormResource.java
src/java/org/apache/fop/render/ps/ResourceHandler.java
src/java/org/apache/fop/render/ps/extensions/PSExtensionHandler.java
src/java/org/apache/fop/render/rtf/TextAttributesConverter.java
src/java/org/apache/fop/render/rtf/rtflib/rtfdoc/ParagraphKeeptogetherContext.java
src/java/org/apache/fop/render/rtf/rtflib/rtfdoc/RtfJforCmd.java
src/java/org/apache/fop/render/rtf/rtflib/rtfdoc/RtfTextrun.java
src/java/org/apache/fop/render/xml/XMLXMLHandler.java
src/java/org/apache/fop/servlet/FopPrintServlet.java
src/java/org/apache/fop/servlet/FopServlet.java
src/java/org/apache/fop/tools/TestConverter.java
src/java/org/apache/fop/tools/anttasks/Fop.java
src/java/org/apache/fop/traits/BorderProps.java
src/java/org/apache/fop/util/ColorUtil.java

index dab1e5ebb4b807e7319751547f98482e4a8b43af..8a393d452e5d1ad0f361c085f25480417ab5d5e0 100644 (file)
@@ -96,7 +96,7 @@
       <property name="severity" value="warning"/>
     </module>
     <module name="ConstantNameCheck">
-      <property name="format" value="^[A-Z](_?[A-Z0-9]+)*$"/>
+      <property name="format" value="^([A-Z](_?[A-Z0-9]+)*)|(log)$"/>
       <property name="severity" value="warning"/>
     </module>
     <module name="LocalFinalVariableNameCheck">
index 6ae6749081ae1c2e0e9a0f3f8877c3b6e84dfd67..20d27c1285b8f6c2826a5f7f706cb3065e0006a4 100644 (file)
@@ -94,8 +94,8 @@ public class ExampleFO2PDF {
             for (java.util.Iterator it = pageSequences.iterator(); it.hasNext();) {
                 PageSequenceResults pageSequenceResults = (PageSequenceResults)it.next();
                 System.out.println("PageSequence "
-                        + (String.valueOf(pageSequenceResults.getID()).length() > 0
-                                ? pageSequenceResults.getID() : "<no id>")
+                        + (String.valueOf(pageSequenceResults.getId()).length() > 0
+                                ? pageSequenceResults.getId() : "<no id>")
                         + " generated " + pageSequenceResults.getPageCount() + " pages.");
             }
             System.out.println("Generated " + foResults.getPageCount() + " pages in total.");
index a9ace7cf059bca51d58284c60b4705e8d00bbf42..0da9ae14c1a5684332d58c758ca3dfba4b33e44c 100644 (file)
@@ -147,8 +147,8 @@ public class MultipleFO2PDF {
                 for (java.util.Iterator it = pageSequences.iterator(); it.hasNext();) {
                     PageSequenceResults pageSequenceResults = (PageSequenceResults)it.next();
                     System.out.println("PageSequence "
-                            + (String.valueOf(pageSequenceResults.getID()).length() > 0
-                                    ? pageSequenceResults.getID() : "<no id>")
+                            + (String.valueOf(pageSequenceResults.getId()).length() > 0
+                                    ? pageSequenceResults.getId() : "<no id>")
                             + " generated " + pageSequenceResults.getPageCount() + " pages.");
                 }
                 System.out.println("Generated " + foResults.getPageCount() + " pages in total.");
index e0da25ff994f55a33f1730f1ab769aa83790ca1c..a2a128be51d08b656a699d828bd21d2e2c7d6fcd 100644 (file)
@@ -30,7 +30,7 @@ import org.apache.commons.logging.LogFactory;
 public abstract class AbstractAFPPainter {
 
     /** Static logging instance */
-    protected static Log log = LogFactory.getLog("org.apache.xmlgraphics.afp");
+    protected static final Log log = LogFactory.getLog("org.apache.xmlgraphics.afp");
 
     /** data stream */
     protected final DataStream dataStream;
index 4b1439da01c9776e2dfe4d72af6083be0a4d8fcd..af679fa25cfb861f3edadc2f0e77b9a27c2306bd 100644 (file)
@@ -45,7 +45,7 @@ public abstract class AbstractAFPObject implements Streamable {
     protected static final byte SF_CLASS = (byte)0xD3;
 
     /** the structure field header */
-    protected static final byte[] SF_HEADER = new byte[] {
+    static final byte[] SF_HEADER = new byte[] {
         0x5A, // Structured field identifier
         0x00, // Length byte 1
         0x10, // Length byte 2
index 241f4a5f96f5cb8d8b8b16f5d0b73365cd96d00e..f94fe8fdb5c7d822d64716c7916dc859bf68c911 100644 (file)
@@ -44,7 +44,7 @@ public class PageSequenceResults {
      *
      * @return   The ID
      */
-    public String getID() {
+    public String getId() {
         return this.id;
     }
 
index ddf2f5198053af5b75ac247cb99818d4456b2091..f94c6845409f98eb5a712426ef4b515a239d2fad 100644 (file)
@@ -38,6 +38,9 @@ import org.apache.fop.traits.BorderProps;
  * Base object for all areas.
  */
 public class Area extends AreaTreeObject implements Serializable {
+
+    private static final long serialVersionUID = 6342888466142626492L;
+
     // stacking directions
     /**
      * Stacking left to right
@@ -134,7 +137,7 @@ public class Area extends AreaTreeObject implements Serializable {
     /**
      * logging instance
      */
-    protected static Log log = LogFactory.getLog(Area.class);
+    protected static final Log log = LogFactory.getLog(Area.class);
 
 
     /**
index e5f6db17bd2016541bf36ea8bc250050d5a3395f..8659b4cca1a5cdbc93d97ab94923320c2a3390c4 100644 (file)
@@ -42,7 +42,7 @@ public class AreaTreeModel {
     protected PageSequence currentPageSequence;
 //    private List offDocumentItems = new java.util.ArrayList();
     /** logger instance */
-    protected static Log log = LogFactory.getLog(AreaTreeModel.class);
+    protected static final Log log = LogFactory.getLog(AreaTreeModel.class);
 
     /**
      * Create a new store pages model
index 31c0e04fcf2df3b1e6a936062cb07f0c1746b6cc..ae7c0a7c3d61359b308770b80bda1a0eb41ae3bc 100644 (file)
@@ -98,7 +98,7 @@ import org.apache.fop.util.XMLUtil;
 public class AreaTreeParser {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(AreaTreeParser.class);
+    protected static final Log log = LogFactory.getLog(AreaTreeParser.class);
 
     private static SAXTransformerFactory tFactory
         = (SAXTransformerFactory)SAXTransformerFactory.newInstance();
index 490476368d93e128dcbcd9d2fa079f39b3b5ea69..b64eff5564bd0016f757adcc20ad5fb2ee44558c 100644 (file)
@@ -25,6 +25,9 @@ package org.apache.fop.area;
  * See fo:region-body definition in the XSL Rec for more information.
  */
 public class BeforeFloat extends BlockParent {
+
+    private static final long serialVersionUID = 4101415711488333380L;
+
     // this is an optional block area that will be rendered
     // as the separator only if there are float areas
     private Block separator = null;
index ecb9c49909de1f9b7cc526621a9d70ae4e3545d1..e37931510593b3c2df27626283a16e704fc52641 100644 (file)
@@ -32,6 +32,9 @@ package org.apache.fop.area;
  * It holds child block areas such as other blocks or lines.
  */
 public class Block extends BlockParent {
+
+    private static final long serialVersionUID = 6843727817993665788L;
+
     /**
      * Normally stacked with other blocks.
      */
index 1be0b785d04f4ad17b8c15699daf27e5d5c982e8..f2cf01cf23645ca0648604f1c605d656c8f205b5 100644 (file)
@@ -27,6 +27,11 @@ import java.util.List;
  */
 public class BlockParent extends Area {
 
+    /**
+     * 
+     */
+    private static final long serialVersionUID = 7076916890348533805L;
+
     // this position is used for absolute position
     // or as an indent
     // this has the size in the block progression dimension
index 60bf1fb22a14f0c07df2e46270f91c669be95a81..032b39f959e8f39bee1f2e6eb7ff1e898395427a 100644 (file)
@@ -25,6 +25,9 @@ package org.apache.fop.area;
  * The block-container creates this area.
  */
 public class BlockViewport extends Block {
+
+    private static final long serialVersionUID = -7840580922580735157L;
+
     // clipping for this viewport
     private boolean clip = false;
     // transform if rotated or absolute
index 7b183a188640a87f6f249a8232aed642323abafc..2dd8a9a7c2212fae51b87f6ba1311edbdc06cbac 100644 (file)
@@ -30,6 +30,9 @@ import org.apache.fop.fo.pagination.RegionBody;
  * See fo:region-body definition in the XSL Rec for more information.
  */
 public class BodyRegion extends RegionReference {
+
+    private static final long serialVersionUID = -1848872997724078080L;
+
     private BeforeFloat beforeFloat;  // optional
     private MainReference mainReference; // mandatory
     private Footnote footnote; // optional
index 099ab38fae8c1137ef23d903bb07b2aa4ceaae89..eb6207c42f9a1eac8909185e36af7e6237cbaa01 100644 (file)
@@ -34,6 +34,8 @@ import org.apache.fop.fo.Constants;
  */
 public class CTM implements Serializable {
 
+    private static final long serialVersionUID = -8743287485623778341L;
+
     private double a, b, c, d, e, f;
 
     private static final CTM CTM_LRTB = new CTM(1, 0, 0, 1, 0, 0);
index a25f5467ea053db00b0202dd74711be74ced93ca..72b9f2ed0669d4fbcf602f3e579914174d9ff6bd 100644 (file)
@@ -27,6 +27,9 @@ package org.apache.fop.area;
  * See fo:region-body definition in the XSL Rec for more information.
  */
 public class Footnote extends BlockParent {
+
+    private static final long serialVersionUID = -7907428219886367161L;
+
     private Block separator = null;
 
     // footnote has an optional separator
index 0e6e6ac309e449b46c56bf1aa9c021006e8c787d..4d622bceca082b466bedbd580aed6e492d5e4506 100644 (file)
@@ -32,12 +32,17 @@ import java.util.List;
  */
 public class LineArea extends Area {
 
+    private static final long serialVersionUID = 7670235908329290684L;
+
     /**
      * this class stores information about line width and potential adjustments
      * that can be used in order to re-compute adjustement and / or indents when a
      * page-number or a page-number-citation is resolved
      */
     private final class LineAdjustingInfo implements Serializable {
+        
+        private static final long serialVersionUID = -6103629976229458273L;
+
         private int lineAlignment;
         private int difference;
         private int availableStretch;
index 51a952ddcfff2ded973bdbcb58a52edea62b3177..70bdfdf916ca5edb47db85f07847bb746c54879a 100644 (file)
@@ -33,6 +33,9 @@ import org.apache.fop.area.Area;
  * Link resolving for resolving internal links.
  */
 public class LinkResolver implements Resolvable, Serializable {
+
+    private static final long serialVersionUID = -7102134165192960718L;
+
     private boolean resolved = false;
     private String idRef;
     private Area area;
index dd53113afaed1cc461454039e819c4adb9254598..87e594169fe724a6289c1b360256081da9564df3 100644 (file)
@@ -31,6 +31,8 @@ import java.util.List;
  */
 public class MainReference extends Area {
 
+    private static final long serialVersionUID = 7635126485620012448L;
+
     private BodyRegion parent;
     private List spanAreas = new java.util.ArrayList();
     private boolean isEmpty = true;
index 5e01923898f8b11ee1972dd3c0c9e2f28c0c77e4..3daf2dafd261a77deef7e410c2e941ef0bca6fcd 100644 (file)
@@ -25,6 +25,9 @@ package org.apache.fop.area;
  * See fo:region-body definition in the XSL Rec for more information.
  */
 public class NormalFlow extends BlockParent {
+
+    private static final long serialVersionUID = -3753538631016929004L;
+
     /**
      * Constructor.
      * @param ipd of Normal flow object
index 4607dfb745bc52071b4c4a961e98f2940c176100..242123baa6f941febcb4822af5ed49cc890523ea 100644 (file)
@@ -47,6 +47,9 @@ import org.apache.fop.layoutmgr.TraitSetter;
  * the top level page and regions.
  */
 public class Page extends AreaTreeObject implements Serializable, Cloneable {
+
+    private static final long serialVersionUID = 6272157047421543866L;
+
     // contains before, start, body, end and after regions
     private RegionViewport regionBefore = null;
     private RegionViewport regionStart = null;
index bccae8754dbb5a69bca5855993a4c3f7fd0c6ded..a4fcb61a809626b115893f5ad31f059301637467 100644 (file)
@@ -87,7 +87,7 @@ public class PageViewport extends AreaTreeObject implements Resolvable, Cloneabl
     /**
      * logging instance
      */
-    protected static Log log = LogFactory.getLog(PageViewport.class);
+    protected static final Log log = LogFactory.getLog(PageViewport.class);
 
     /**
      * Create a page viewport.
index 200f0bff3dedaea1391733783ca419352b0d181e..0aec7a3958730d56f25d861d606f672fff03ab84 100644 (file)
@@ -31,13 +31,14 @@ import org.apache.fop.fo.pagination.Region;
  */
 public class RegionReference extends Area implements Cloneable {
 
+    private static final long serialVersionUID = -298980963268244238L;
+
     /** Reference to the region FO. */
     //protected Region regionFO;
     private int regionClass;
     private String regionName;
     private CTM ctm;
 
-
     // the list of block areas from the static flow
     private ArrayList blocks = new ArrayList();
 
index d55ecefbb3936c3c22e680926898479d8ff9d2c5..0654c5f6b171d7dcf7ee17ca665be47c429b5721 100644 (file)
@@ -30,6 +30,9 @@ import java.util.HashMap;
  * in the fo:region-body description in the XSL Recommendation.
  */
 public class RegionViewport extends Area implements Cloneable {
+
+    private static final long serialVersionUID = 505781815165102572L;
+
     // this rectangle is relative to the page
     private RegionReference regionReference;
     private Rectangle2D viewArea;
index 69b37f4b3b8920a832e67d524842d74f4546b81a..e2ef904327d396825c148db8f66b3d871299a90d 100644 (file)
@@ -30,6 +30,9 @@ import java.util.List;
  * See fo:region-body definition in the XSL Rec for more information.
  */
 public class Span extends Area {
+
+    private static final long serialVersionUID = -5551430053660081549L;
+
     // the list of flow reference areas in this span area
     private List flowAreas;
     private int colCount;
index 42c5462e445dd4fefe8a37125f423800a5d749ae..4ad03b4689b4d2630855eb14d6a566e0077636b9 100644 (file)
@@ -36,6 +36,8 @@ import org.apache.fop.util.ColorUtil;
  */
 public final class Trait implements Serializable {
 
+    private static final long serialVersionUID = 3234280285391611437L;
+
     private Trait() {
     }
 
@@ -313,6 +315,8 @@ public final class Trait implements Serializable {
      */
     public static class InternalLink implements Serializable {
 
+        private static final long serialVersionUID = -8993505060996723039L;
+
         /** The unique key of the PageViewport. */
         private String pvKey;
 
@@ -447,6 +451,8 @@ public final class Trait implements Serializable {
      */
     public static class ExternalLink implements Serializable {
 
+        private static final long serialVersionUID = -3720707599232620946L;
+
         private String destination;
         private boolean newWindow;
 
@@ -520,6 +526,8 @@ public final class Trait implements Serializable {
      */
     public static class Background implements Serializable {
 
+        private static final long serialVersionUID = 8452078676273242870L;
+
         /** The background color if any. */
         private Color color = null;
 
index b50f53a24814970fe88022cb8169d2c18f99ab03..08997a8179ea8dd511b91b4381654206ed0fa89d 100644 (file)
@@ -24,6 +24,8 @@ package org.apache.fop.area.inline;
  */
 public abstract class AbstractTextArea extends InlineParent {
 
+    private static final long serialVersionUID = -1246306443569094371L;
+
     /**
      * this class stores information about spaces and potential adjustments
      * that can be used in order to re-compute adjustments when a
@@ -31,6 +33,8 @@ public abstract class AbstractTextArea extends InlineParent {
      */
     protected class TextAdjustingInfo extends InlineAdjustingInfo {
 
+        private static final long serialVersionUID = -2412095162983479947L;
+
         /** difference between the optimal width of a space
          * and the default width of a space according to the font
          * (this is equivalent to the property word-spacing.optimum)
index ec6dbbcfb749e9fee78b012b3279d0595779cb28..8ad663111f37fb546a29d4366030ae22756eda2f 100644 (file)
@@ -25,6 +25,8 @@ package org.apache.fop.area.inline;
  */
 public class Anchor extends InlineArea {
 
+    private static final long serialVersionUID = 5227798744787823499L;
+
     // has a keep with adjacent area
     // has reference to associated footnote or float out-of-line area
 
index 98c0aaef05a4f55bc6c8ca85b92d7094a1a72695..f6dbf3a1574bfc59d53077186cb2cdf4f35795c5 100644 (file)
@@ -32,6 +32,9 @@ import java.util.ArrayList;
  * This allows an inline area to have blocks as children.
  */
 public class Container extends Area {
+
+    private static final long serialVersionUID = 5256423939348189260L;
+
     /**
      * The list of block areas stacked inside this container
      */
index 7df62e79bce3692b7c48b7636ccc4241fb2c0c7b..06d0d3aa340a342553a37386447a697ec60d4206 100644 (file)
@@ -34,6 +34,9 @@ import java.util.Iterator;
  * this as a normal inline parent.
  */
 public class FilledArea extends InlineParent {
+
+    private static final long serialVersionUID = 8586584705587017474L;
+
     private int unitWidth;
 
     /**
index 7d802db499c3eef3fb2ca22f53da5493f72303a5..baf3c183b683c27b101688df2e79a854c64f7ca7 100644 (file)
@@ -27,6 +27,9 @@ import org.apache.fop.area.Area;
  * The url of the image is used as a key to reference the image cache.
  */
 public class Image extends Area {
+
+    private static final long serialVersionUID = 4800834714349695386L;
+
     private String url;
     
     /**
index 30afc149b327a49e5acc9339cfade476517490f1..0889d1dad9ed090ea462978f4f31c094dfe991c4 100644 (file)
@@ -32,12 +32,17 @@ import org.apache.fop.area.Trait;
  */
 public class InlineArea extends Area {
 
+    private static final long serialVersionUID = -8940066479810170980L;
+
     /**
      * this class stores information about potential adjustments
      * that can be used in order to re-compute adjustments when a
      * page-number or a page-number-citation is resolved
      */
     protected class InlineAdjustingInfo implements Serializable {
+
+        private static final long serialVersionUID = -5601387735459712149L;
+
         /** stretch of the inline area */
         protected int availableStretch;
         /** shrink of the inline area */
index e580b166719c5d7954d796c4b4fadb74c524c276..fe024ac0f7a459fb35f978ad4f7b73372a91e036 100644 (file)
@@ -29,6 +29,9 @@ import org.apache.fop.area.Block;
  */
 public class InlineBlockParent extends InlineArea {
 
+
+    private static final long serialVersionUID = -3661746143321407377L;
+
     /**
      * The list of inline areas added to this inline parent.
      */
index cbc5b578c27ab7067ac10b93de87c9273cc81253..5d43691037d3b413dedd0dcf5be4c11c4106cce4 100644 (file)
@@ -29,6 +29,9 @@ import java.util.ArrayList;
  * This is an inline area that can have other inlines as children.
  */
 public class InlineParent extends InlineArea {
+
+    private static final long serialVersionUID = -3047168298770354813L;
+
     /**
      * The list of inline areas added to this inline parent.
      */
index cd7b2e9f3221664cca36deec042a1fcce701f6f1..ff8ba12b03d7d4a171f4f2cb3a96c63d60914d11 100644 (file)
@@ -32,6 +32,9 @@ public class Leader extends InlineArea {
     // if space replaced with a space
     // otherwise this is a holder for a line
 
+
+    private static final long serialVersionUID = -8011373048313956301L;
+
     private int ruleStyle = Constants.EN_SOLID;
     private int ruleThickness = 1000;
 
index 9a24de0d3ca1e7ba8f7c56e00570cb8910a58b95..bf683bb2286d7ffeead7a3b165d0c96243f0de6c 100644 (file)
@@ -25,4 +25,6 @@ package org.apache.fop.area.inline;
  */
 public class Space extends InlineArea {
 
+    private static final long serialVersionUID = -8748265505356839796L;
+
 }
index 202408debac6de47737d7f0a3298a6e6d7cfff9e..6f08637895b05d4923608e140bc0ba1c3870e9d7 100644 (file)
@@ -24,6 +24,8 @@ package org.apache.fop.area.inline;
  */
 public class SpaceArea extends InlineArea {
 
+    private static final long serialVersionUID = 2218803009825411416L;
+
     /**
      * The space for this space area
      */
index f736b894c6ef1dc2d9442678361fc76a3925a910..8fb9455add6c8a6a47c54fbd9435b8e56c7d06ef 100644 (file)
@@ -24,6 +24,8 @@ package org.apache.fop.area.inline;
  */
 public class TextArea extends AbstractTextArea {
 
+    private static final long serialVersionUID = 7315900267242540809L;
+
     /**
      * Create a text inline area
      */
index 43a995670a233183ecc77523b557885bfb8f3ca9..3457ad69804690f653547e988db523e6e552d832 100644 (file)
@@ -32,6 +32,9 @@ import java.util.List;
  */
 public class UnresolvedPageNumber extends TextArea implements Resolvable {
 
+
+    private static final long serialVersionUID = -1758090835371647980L;
+
     private boolean resolved = false;
     private String pageIDRef;
     private String text;
index 489f5afda1ed4e416a17534bde2037226f385ddb..206a965e2bb9037128e4b44f8e51cd685f1d41fb 100644 (file)
@@ -32,6 +32,9 @@ import java.util.HashMap;
  * holds the area and positions it.
  */
 public class Viewport extends InlineArea {
+
+    private static final long serialVersionUID = 813338534627918689L;
+
     // contents could be container, foreign object or image
     private Area content;
     // clipping for the viewport
index 4737d81d33775f0f3a328c539dcb1337641c5c0a..8bb6888ce3164e7ab863847240771e26179e033d 100644 (file)
@@ -24,6 +24,8 @@ package org.apache.fop.area.inline;
  */
 public class WordArea extends InlineArea {
 
+    private static final long serialVersionUID = 6444644662158970942L;
+
     /** The text for this word area */
     protected String word;
 
index 2a5c0d272f2f1713306287fe34353a2f572a570b..eff7b76f0383d7221eb2e20bee2d4e47b98112b1 100644 (file)
@@ -89,23 +89,23 @@ public class CommandLineOptions {
     /* user configuration file */
     private File userConfigFile = null;
     /* input fo file */
-    private File fofile = null;
+    private File foFile = null;
     /* xsltfile (xslt transformation as input) */
-    private File xsltfile = null;
+    private File xsltFile = null;
     /* xml file (xslt transformation as input) */
-    private File xmlfile = null;
+    private File xmlFile = null;
     /* area tree input file */
-    private File areatreefile = null;
+    private File areatreeFile = null;
     /* intermediate format input file */
-    private File iffile = null;
+    private File ifFile = null;
     /* area tree input file */
-    private File imagefile = null;
+    private File imageFile = null;
     /* output file */
-    private File outfile = null;
+    private File outFile = null;
     /* input mode */
-    private int inputmode = NOT_SET;
+    private int inputMode = NOT_SET;
     /* output mode */
-    private String outputmode = null;
+    private String outputMode = null;
     /* true if System.in (stdin) should be used for the input file */
     private boolean useStdIn = false;
     /* true if System.out (stdout) should be used for the output file */
@@ -194,7 +194,7 @@ public class CommandLineOptions {
 
         inputHandler = createInputHandler();
 
-        if (MimeConstants.MIME_FOP_AWT_PREVIEW.equals(outputmode)) {
+        if (MimeConstants.MIME_FOP_AWT_PREVIEW.equals(outputMode)) {
             //set the system look&feel for the preview dialog
             try {
                 UIManager.setLookAndFeel(
@@ -207,7 +207,7 @@ public class CommandLineOptions {
             renderer.setRenderable(inputHandler); //set before user agent!
             renderer.setUserAgent(foUserAgent);
             foUserAgent.setRendererOverride(renderer);
-        } else if (MimeConstants.MIME_FOP_AREA_TREE.equals(outputmode)
+        } else if (MimeConstants.MIME_FOP_AREA_TREE.equals(outputMode)
                && mimicRenderer != null) {
             // render from FO to Intermediate Format
             Renderer targetRenderer = foUserAgent.getRendererFactory().createRenderer(
@@ -220,7 +220,7 @@ public class CommandLineOptions {
 
             //Make sure the prepared XMLRenderer is used
             foUserAgent.setRendererOverride(xmlRenderer);
-        } else if (MimeConstants.MIME_FOP_IF.equals(outputmode)
+        } else if (MimeConstants.MIME_FOP_IF.equals(outputMode)
                 && mimicRenderer != null) {
             // render from FO to Intermediate Format
             IFSerializer serializer = new IFSerializer();
@@ -447,7 +447,7 @@ public class CommandLineOptions {
             if (isSystemInOutFile(filename)) {
                 this.useStdIn = true;
             } else {
-                fofile = new File(filename);
+                foFile = new File(filename);
             }
             return 1;
         }
@@ -460,7 +460,7 @@ public class CommandLineOptions {
             throw new FOPException("you must specify the stylesheet "
                             + "file for the '-xsl' option");
         } else {
-            xsltfile = new File(args[i + 1]);
+            xsltFile = new File(args[i + 1]);
             return 1;
         }
     }
@@ -476,7 +476,7 @@ public class CommandLineOptions {
             if (isSystemInOutFile(filename)) {
                 this.useStdIn = true;
             } else {
-                xmlfile = new File(filename);
+                xmlFile = new File(filename);
             }
             return 1;
         }
@@ -508,7 +508,7 @@ public class CommandLineOptions {
         if (isSystemInOutFile(filename)) {
             this.useStdOut = true;
         } else {
-            outfile = new File(filename);
+            outFile = new File(filename);
         }
     }
 
@@ -701,16 +701,16 @@ public class CommandLineOptions {
     }
 
     private int parseUnknownOption(String[] args, int i) throws FOPException {
-        if (inputmode == NOT_SET) {
-            inputmode = FO_INPUT;
+        if (inputMode == NOT_SET) {
+            inputMode = FO_INPUT;
             String filename = args[i];
             if (isSystemInOutFile(filename)) {
                 this.useStdIn = true;
             } else {
-                fofile = new File(filename);
+                foFile = new File(filename);
             }
-        } else if (outputmode == null) {
-            outputmode = MimeConstants.MIME_PDF;
+        } else if (outputMode == null) {
+            outputMode = MimeConstants.MIME_PDF;
             setOutputFile(args[i]);
         } else {
             throw new FOPException("Don't know what to do with "
@@ -765,7 +765,7 @@ public class CommandLineOptions {
             if (isSystemInOutFile(filename)) {
                 this.useStdIn = true;
             } else {
-                areatreefile = new File(filename);
+                areatreeFile = new File(filename);
             }
             return 1;
         }
@@ -781,7 +781,7 @@ public class CommandLineOptions {
             if (isSystemInOutFile(filename)) {
                 this.useStdIn = true;
             } else {
-                iffile = new File(filename);
+                ifFile = new File(filename);
             }
             return 1;
         }
@@ -797,7 +797,7 @@ public class CommandLineOptions {
             if (isSystemInOutFile(filename)) {
                 this.useStdIn = true;
             } else {
-                imagefile = new File(filename);
+                imageFile = new File(filename);
             }
             return 1;
         }
@@ -867,8 +867,8 @@ public class CommandLineOptions {
     }
 
     private void setOutputMode(String mime) throws FOPException {
-        if (outputmode == null) {
-            outputmode = mime;
+        if (outputMode == null) {
+            outputMode = mime;
         } else {
             throw new FOPException("you can only set one output method");
         }
@@ -896,8 +896,8 @@ public class CommandLineOptions {
     }
 
     private void setInputFormat(int format) throws FOPException {
-        if (inputmode == NOT_SET || inputmode == format) {
-            inputmode = format;
+        if (inputMode == NOT_SET || inputMode == format) {
+            inputMode = format;
         } else {
             throw new FOPException("Only one input mode can be specified!");
         }
@@ -907,100 +907,100 @@ public class CommandLineOptions {
      * checks whether all necessary information has been given in a consistent way
      */
     private void checkSettings() throws FOPException, FileNotFoundException {
-        if (inputmode == NOT_SET) {
+        if (inputMode == NOT_SET) {
             throw new FOPException("No input file specified");
         }
 
-        if (outputmode == null) {
+        if (outputMode == null) {
             throw new FOPException("No output file specified");
         }
 
-        if ((outputmode.equals(MimeConstants.MIME_FOP_AWT_PREVIEW)
-                || outputmode.equals(MimeConstants.MIME_FOP_PRINT))
-                    && outfile != null) {
+        if ((outputMode.equals(MimeConstants.MIME_FOP_AWT_PREVIEW)
+                || outputMode.equals(MimeConstants.MIME_FOP_PRINT))
+                    && outFile != null) {
             throw new FOPException("Output file may not be specified "
                     + "for AWT or PRINT output");
         }
 
-        if (inputmode == XSLT_INPUT) {
+        if (inputMode == XSLT_INPUT) {
             // check whether xml *and* xslt file have been set
-            if (xmlfile == null && !this.useStdIn) {
+            if (xmlFile == null && !this.useStdIn) {
                 throw new FOPException("XML file must be specified for the transform mode");
             }
-            if (xsltfile == null) {
+            if (xsltFile == null) {
                 throw new FOPException("XSLT file must be specified for the transform mode");
             }
 
             // warning if fofile has been set in xslt mode
-            if (fofile != null) {
+            if (foFile != null) {
                 log.warn("Can't use fo file with transform mode! Ignoring.\n"
                                        + "Your input is " + "\n xmlfile: "
-                                       + xmlfile.getAbsolutePath()
+                                       + xmlFile.getAbsolutePath()
                                        + "\nxsltfile: "
-                                       + xsltfile.getAbsolutePath()
+                                       + xsltFile.getAbsolutePath()
                                        + "\n  fofile: "
-                                       + fofile.getAbsolutePath());
+                                       + foFile.getAbsolutePath());
             }
-            if (xmlfile != null && !xmlfile.exists()) {
+            if (xmlFile != null && !xmlFile.exists()) {
                 throw new FileNotFoundException("Error: xml file "
-                                                + xmlfile.getAbsolutePath()
+                                                + xmlFile.getAbsolutePath()
                                                 + " not found ");
             }
-            if (!xsltfile.exists()) {
+            if (!xsltFile.exists()) {
                 throw new FileNotFoundException("Error: xsl file "
-                                                + xsltfile.getAbsolutePath()
+                                                + xsltFile.getAbsolutePath()
                                                 + " not found ");
             }
 
-        } else if (inputmode == FO_INPUT) {
-            if (outputmode.equals(MimeConstants.MIME_XSL_FO)) {
+        } else if (inputMode == FO_INPUT) {
+            if (outputMode.equals(MimeConstants.MIME_XSL_FO)) {
                 throw new FOPException(
                         "FO output mode is only available if you use -xml and -xsl");
             }
-            if (fofile != null && !fofile.exists()) {
+            if (foFile != null && !foFile.exists()) {
                 throw new FileNotFoundException("Error: fo file "
-                                                + fofile.getAbsolutePath()
+                                                + foFile.getAbsolutePath()
                                                 + " not found ");
             }
-        } else if (inputmode == AREATREE_INPUT) {
-            if (outputmode.equals(MimeConstants.MIME_XSL_FO)) {
+        } else if (inputMode == AREATREE_INPUT) {
+            if (outputMode.equals(MimeConstants.MIME_XSL_FO)) {
                 throw new FOPException(
                         "FO output mode is only available if you use -xml and -xsl");
-            } else if (outputmode.equals(MimeConstants.MIME_FOP_AREA_TREE)) {
+            } else if (outputMode.equals(MimeConstants.MIME_FOP_AREA_TREE)) {
                 throw new FOPException(
                         "Area Tree Output is not available if Area Tree is used as input!");
             }
-            if (areatreefile != null && !areatreefile.exists()) {
+            if (areatreeFile != null && !areatreeFile.exists()) {
                 throw new FileNotFoundException("Error: area tree file "
-                                              + areatreefile.getAbsolutePath()
+                                              + areatreeFile.getAbsolutePath()
                                               + " not found ");
             }
-        } else if (inputmode == IF_INPUT) {
-            if (outputmode.equals(MimeConstants.MIME_XSL_FO)) {
+        } else if (inputMode == IF_INPUT) {
+            if (outputMode.equals(MimeConstants.MIME_XSL_FO)) {
                 throw new FOPException(
                         "FO output mode is only available if you use -xml and -xsl");
-            } else if (outputmode.equals(MimeConstants.MIME_FOP_AREA_TREE)) {
+            } else if (outputMode.equals(MimeConstants.MIME_FOP_AREA_TREE)) {
                 throw new FOPException(
                     "Area Tree Output is not available if Intermediate Format"
                     + " is used as input!");
-            } else if (outputmode.equals(MimeConstants.MIME_FOP_IF)) {
+            } else if (outputMode.equals(MimeConstants.MIME_FOP_IF)) {
                 throw new FOPException(
                     "Intermediate Output is not available if Intermediate Format"
                     + " is used as input!");
             }
-            if (iffile != null && !iffile.exists()) {
+            if (ifFile != null && !ifFile.exists()) {
                 throw new FileNotFoundException("Error: intermediate format file "
-                                              + iffile.getAbsolutePath()
+                                              + ifFile.getAbsolutePath()
                                               + " not found ");
             }
-        } else if (inputmode == IMAGE_INPUT) {
-            if (outputmode.equals(MimeConstants.MIME_XSL_FO)) {
+        } else if (inputMode == IMAGE_INPUT) {
+            if (outputMode.equals(MimeConstants.MIME_XSL_FO)) {
                 throw new FOPException(
                         "FO output mode is only available if you use -xml and -xsl");
             }
-            if (imagefile != null && !imagefile.exists()) {
+            if (imageFile != null && !imageFile.exists()) {
                 throw new FileNotFoundException("Error: image file "
-                                              + imagefile.getAbsolutePath()
+                                              + imageFile.getAbsolutePath()
                                               + " not found ");
             }
         }
@@ -1027,13 +1027,13 @@ public class CommandLineOptions {
      * @throws FOPException for invalid output formats
      */
     protected String getOutputFormat() throws FOPException {
-        if (outputmode == null) {
+        if (outputMode == null) {
             throw new FOPException("Renderer has not been set!");
         }
-        if (outputmode.equals(MimeConstants.MIME_FOP_AREA_TREE)) {
+        if (outputMode.equals(MimeConstants.MIME_FOP_AREA_TREE)) {
             renderingOptions.put("fineDetail", isCoarseAreaXml());
         }
-        return outputmode;
+        return outputMode;
     }
 
     /**
@@ -1042,21 +1042,21 @@ public class CommandLineOptions {
      * @throws IllegalArgumentException if invalid/missing parameters
      */
     private InputHandler createInputHandler() {
-        switch (inputmode) {
+        switch (inputMode) {
             case FO_INPUT:
-                return new InputHandler(fofile);
+                return new InputHandler(foFile);
             case AREATREE_INPUT:
-                return new AreaTreeInputHandler(areatreefile);
+                return new AreaTreeInputHandler(areatreeFile);
             case IF_INPUT:
-                return new IFInputHandler(iffile);
+                return new IFInputHandler(ifFile);
             case XSLT_INPUT:
-                InputHandler handler = new InputHandler(xmlfile, xsltfile, xsltParams);
+                InputHandler handler = new InputHandler(xmlFile, xsltFile, xsltParams);
                 if (useCatalogResolver) {
                     handler.createCatalogResolver(foUserAgent);
                 }
                 return handler;
             case IMAGE_INPUT:
-                return new ImageInputHandler(imagefile, xsltfile, xsltParams);
+                return new ImageInputHandler(imageFile, xsltFile, xsltParams);
             default:
                 throw new IllegalArgumentException("Error creating InputHandler object.");
         }
@@ -1074,24 +1074,24 @@ public class CommandLineOptions {
      * Returns the XSL-FO file if set.
      * @return the XSL-FO file, null if not set
      */
-    public File getFOFile() {
-        return fofile;
+    public File getFoFile() {
+        return foFile;
     }
 
     /**
      * Returns the input XML file if set.
      * @return the input XML file, null if not set
      */
-    public File getXMLFile() {
-        return xmlfile;
+    public File getXmlFile() {
+        return xmlFile;
     }
 
     /**
      * Returns the stylesheet to be used for transformation to XSL-FO.
      * @return stylesheet
      */
-    public File getXSLFile() {
-        return xsltfile;
+    public File getXslFile() {
+        return xsltFile;
     }
 
     /**
@@ -1099,7 +1099,7 @@ public class CommandLineOptions {
      * @return the output file
      */
     public File getOutputFile() {
-        return outfile;
+        return outFile;
     }
 
     /**
@@ -1139,13 +1139,13 @@ public class CommandLineOptions {
      * @return either the fofile or the xmlfile
      */
     public File getInputFile() {
-        switch (inputmode) {
+        switch (inputMode) {
         case FO_INPUT:
-            return fofile;
+            return foFile;
         case XSLT_INPUT:
-            return xmlfile;
+            return xmlFile;
         default:
-            return fofile;
+            return foFile;
         }
     }
 
@@ -1270,7 +1270,7 @@ public class CommandLineOptions {
      */
     private void dumpConfiguration() {
         log.info("Input mode: ");
-        switch (inputmode) {
+        switch (inputMode) {
         case NOT_SET:
             log.info("not set");
             break;
@@ -1279,7 +1279,7 @@ public class CommandLineOptions {
             if (isInputFromStdIn()) {
                 log.info("fo input file: from stdin");
             } else {
-                log.info("fo input file: " + fofile.toString());
+                log.info("fo input file: " + foFile.toString());
             }
             break;
         case XSLT_INPUT:
@@ -1287,16 +1287,16 @@ public class CommandLineOptions {
             if (isInputFromStdIn()) {
                 log.info("xml input file: from stdin");
             } else {
-                log.info("xml input file: " + xmlfile.toString());
+                log.info("xml input file: " + xmlFile.toString());
             }
-            log.info("xslt stylesheet: " + xsltfile.toString());
+            log.info("xslt stylesheet: " + xsltFile.toString());
             break;
         case AREATREE_INPUT:
             log.info("AT ");
             if (isInputFromStdIn()) {
                 log.info("area tree input file: from stdin");
             } else {
-                log.info("area tree input file: " + areatreefile.toString());
+                log.info("area tree input file: " + areatreeFile.toString());
             }
             break;
         case IF_INPUT:
@@ -1304,7 +1304,7 @@ public class CommandLineOptions {
             if (isInputFromStdIn()) {
                 log.info("intermediate input file: from stdin");
             } else {
-                log.info("intermediate input file: " + iffile.toString());
+                log.info("intermediate input file: " + ifFile.toString());
             }
             break;
         case IMAGE_INPUT:
@@ -1312,28 +1312,28 @@ public class CommandLineOptions {
             if (isInputFromStdIn()) {
                 log.info("image input file: from stdin");
             } else {
-                log.info("image input file: " + imagefile.toString());
+                log.info("image input file: " + imageFile.toString());
             }
             break;
         default:
             log.info("unknown input type");
         }
         log.info("Output mode: ");
-        if (outputmode == null) {
+        if (outputMode == null) {
             log.info("not set");
-        } else if (MimeConstants.MIME_FOP_AWT_PREVIEW.equals(outputmode)) {
+        } else if (MimeConstants.MIME_FOP_AWT_PREVIEW.equals(outputMode)) {
             log.info("awt on screen");
-            if (outfile != null) {
+            if (outFile != null) {
                 log.error("awt mode, but outfile is set:");
-                log.error("out file: " + outfile.toString());
+                log.error("out file: " + outFile.toString());
             }
-        } else if (MimeConstants.MIME_FOP_PRINT.equals(outputmode)) {
+        } else if (MimeConstants.MIME_FOP_PRINT.equals(outputMode)) {
             log.info("print directly");
-            if (outfile != null) {
+            if (outFile != null) {
                 log.error("print mode, but outfile is set:");
-                log.error("out file: " + outfile.toString());
+                log.error("out file: " + outFile.toString());
             }
-        } else if (MimeConstants.MIME_FOP_AREA_TREE.equals(outputmode)) {
+        } else if (MimeConstants.MIME_FOP_AREA_TREE.equals(outputMode)) {
             log.info("area tree");
             if (mimicRenderer != null) {
               log.info("mimic renderer: " + mimicRenderer);
@@ -1341,17 +1341,17 @@ public class CommandLineOptions {
             if (isOutputToStdOut()) {
                 log.info("output file: to stdout");
             } else {
-                log.info("output file: " + outfile.toString());
+                log.info("output file: " + outFile.toString());
             }
-        } else if (MimeConstants.MIME_FOP_IF.equals(outputmode)) {
+        } else if (MimeConstants.MIME_FOP_IF.equals(outputMode)) {
             log.info("intermediate format");
-            log.info("output file: " + outfile.toString());
+            log.info("output file: " + outFile.toString());
         } else {
-            log.info(outputmode);
+            log.info(outputMode);
             if (isOutputToStdOut()) {
                 log.info("output file: to stdout");
             } else {
-                log.info("output file: " + outfile.toString());
+                log.info("output file: " + outFile.toString());
             }
         }
 
index 137ca94fa5f15d3e18189353166334f50dea3ee3..00eaee8c87b97525836b440878d84bbdeeb589a0 100644 (file)
@@ -62,7 +62,7 @@ public class LengthBase implements PercentBase {
     /**
      * logging instance
      */
-    protected static Log log = LogFactory.getLog(LengthBase.class);
+    protected static final Log log = LogFactory.getLog(LengthBase.class);
 
     /**
      * The FO for which this property is to be calculated.
index a6be832f00214dbd5d5187935cd3e75dbc7477e4..c9c04aa4e25aefd8aba5804eeae772defd64cd43 100644 (file)
@@ -69,7 +69,7 @@ public abstract class FONode implements Cloneable {
     protected Locator locator;
 
     /** Logger for fo-tree related messages **/
-    protected static Log log = LogFactory.getLog(FONode.class);
+    protected static final Log log = LogFactory.getLog(FONode.class);
 
     /**
      * Base constructor
index 5db11f731b66957ffcb493cf6883132696888bb7..c78421cbe6cd1ab40486ef390704b242d84cda74 100644 (file)
@@ -619,7 +619,7 @@ public class FOText extends FONode implements CharSequence {
     }
 
     /** @return the baseline-shift property */
-    public Length getBaseLineShift() {
+    public Length getBaselineShift() {
         return baselineShift;
     }
 
index 22474129467a36c9786bdc7f3e3b7723c6a3daf9..6dcc313857c77c99719c508e302fbe572ca80ebf 100644 (file)
@@ -33,6 +33,8 @@ import org.xml.sax.SAXException;
  */
 public class XMPMetadata implements ExtensionAttachment, Serializable, XMLizable {
 
+    private static final long serialVersionUID = 591347206217931578L;
+
     /** The category URI for this extension attachment. */
     public static final String CATEGORY = XMPConstants.XMP_NAMESPACE;
 
index 29e8faac5fe5aafd4bd5efd2904799f51b8ef298..a04f96a3c6a1b15341f5ca1be189338f5d1c62bc 100644 (file)
@@ -36,7 +36,7 @@ import org.apache.fop.fo.Constants;
 public class Property {
 
     /** Logger for all property classes */
-    protected static Log log = LogFactory.getLog(PropertyMaker.class);
+    protected static final Log log = LogFactory.getLog(PropertyMaker.class);
 
     /**
      * The original specified value for properties which inherit
@@ -197,6 +197,6 @@ public class Property {
         if (obj != this) {
             return obj.toString();
         }
-        return null;
+        return "";
     }
 }
index a7b840fcc6ae76cf4c87471f94c7c2263a456b6f..e36b5ff68d2123c3792bc90d11bb73733cc117c2 100644 (file)
@@ -41,7 +41,7 @@ import org.apache.commons.logging.LogFactory;
 public class FontInfo {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(FontInfo.class);
+    protected static final Log log = LogFactory.getLog(FontInfo.class);
 
     /** Map containing fonts that have been used */
     private Map/*<String,FontMetrics>*/ usedFonts = null; //(String = font key)
index 9a11f84bc66269c81ab0189708ab9ad98351fccb..911e576467a1ccdc6a68cc191ec7335bd31f94e5 100644 (file)
@@ -44,7 +44,7 @@ import org.apache.fop.util.LogUtil;
  */
 public class FontInfoConfigurator {
     /** logger instance */
-    protected static Log log = LogFactory.getLog(FontInfoConfigurator.class);
+    protected static final Log log = LogFactory.getLog(FontInfoConfigurator.class);
 
     private Configuration cfg;
     private FontManager fontManager;
index 93d4ae03f280217b9093bd8b1d526525027e8ebd..02c09a1a1054104006ee646f4f57c8375fef14f3 100644 (file)
@@ -40,7 +40,7 @@ import org.apache.fop.fonts.type1.Type1FontLoader;
 public abstract class FontLoader {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(FontLoader.class);
+    protected static final Log log = LogFactory.getLog(FontLoader.class);
 
     /** URI representing the font file */
     protected String fontFileURI = null;
index e5e8ca524d35eabfea79a016ee6b5e209271a4ca..bc294057baae65d43063829ff48732f05dcf4688 100644 (file)
@@ -173,7 +173,7 @@ public class PFMReader extends AbstractFontReader {
     public void preview(PFMFile pfm) {
         if (log != null && log.isInfoEnabled()) {
             log.info("Font: " + pfm.getWindowsName());
-            log.info("Name: " + pfm.getPostscriptName());
+            log.info("Name: " + pfm.getPostScriptName());
             log.info("CharSet: " + pfm.getCharSetName());
             log.info("CapHeight: " + pfm.getCapHeight());
             log.info("XHeight: " + pfm.getXHeight());
@@ -217,7 +217,7 @@ public class PFMReader extends AbstractFontReader {
 
         Element el = doc.createElement("font-name");
         root.appendChild(el);
-        el.appendChild(doc.createTextNode(pfm.getPostscriptName()));
+        el.appendChild(doc.createTextNode(pfm.getPostScriptName()));
 
         // Currently unused.
         // String s = pfm.getPostscriptName();
index 313ad04a83f7140500d8f0b3ee69d357b425a4a1..e018bb2c2ff10a3e51a427e551de9980c6f5cf5f 100644 (file)
@@ -35,7 +35,7 @@ public class FontSubstitutions extends java.util.ArrayList/*<Substitutions>*/ {
     private static final long serialVersionUID = -9173104935431899722L;
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(FontSubstitutions.class);
+    protected static final Log log = LogFactory.getLog(FontSubstitutions.class);
 
     /**
      * Adjusts a given fontInfo using this font substitution catalog
index 29ae0556d6404469969316ff5b22682d72ab2643..34e04eb67b2ec4b9313b802a7c4d5ec4b0f9db0e 100644 (file)
@@ -30,7 +30,7 @@ import org.apache.commons.logging.LogFactory;
 public class FontWeightRange {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog("org.apache.fop.render.fonts");
+    protected static final Log log = LogFactory.getLog("org.apache.fop.render.fonts");
 
     /**
      * Returns an <code>FontWeightRange</code> object holding the
index d1a3d79ff76baf0fcc344333557211e33542a636..b5faddac1719953cbd782e05c64d037fc1a85017 100644 (file)
@@ -38,7 +38,7 @@ public class PFMFile {
 
     // Header stuff
     private String windowsName;
-    private String postscriptName;
+    private String postScriptName;
     private short dfItalic;
     private int dfWeight;
     private short dfCharSet;
@@ -164,7 +164,7 @@ public class PFMFile {
 
         inStream.reset();
         inStream.skip(driverInfoOffset);
-        postscriptName = inStream.readString();
+        postScriptName = inStream.readString();
 
         if (extMetricsOffset != 0) {
             inStream.reset();
@@ -279,8 +279,8 @@ public class PFMFile {
      *
      * @return The Postscript name.
      */
-    public String getPostscriptName() {
-        return postscriptName;
+    public String getPostScriptName() {
+        return postScriptName;
     }
 
     /**
@@ -304,7 +304,7 @@ public class PFMFile {
         case 0:
             return "WinAnsi"; // AKA ISOAdobe
         case 2:
-            if ("Symbol".equals(getPostscriptName())) {
+            if ("Symbol".equals(getPostScriptName())) {
                 return "Symbol";
             }
             break;
index d5036de7ba4c7962a4f893d32a25ca391adddce2..0c0c6187c79b40a7004e091cdde29c82e88d4283 100644 (file)
@@ -226,8 +226,8 @@ public class Type1FontLoader extends FontLoader {
             names.add(afm.getFamilyName());
             returnFont.setFamilyNames(names);
         } else {
-            returnFont.setFontName(pfm.getPostscriptName());
-            String fullName = pfm.getPostscriptName();
+            returnFont.setFontName(pfm.getPostScriptName());
+            String fullName = pfm.getPostScriptName();
             fullName = fullName.replace('-', ' '); //Hack! Try to emulate full name
             returnFont.setFullName(fullName); //emulate afm.getFullName()
             Set names = new java.util.HashSet();
index d1448abdc192a8b5ad80db460d16b9740e6f7403..0df1bd92576afbb64f476c2eaf714bd36d772e49 100644 (file)
@@ -29,6 +29,8 @@ import java.io.Serializable;
  */
 public class ByteVector implements Serializable {
 
+    private static final long serialVersionUID = 1554572867863466772L;
+
     /**
      * Capacity increment size
      */
index 0ba42362b32ff2eaea5aa23c384c4eb9757a7b82..97a63392e85e1428acb16a81da2345d87dbed203 100644 (file)
@@ -29,6 +29,8 @@ import java.io.Serializable;
  */
 public class CharVector implements Cloneable, Serializable {
 
+    private static final long serialVersionUID = 4263472982169004048L;
+
     /**
      * Capacity increment size
      */
index ca876e1ece834d535e059e71b4a1685b13e6b9e8..ae33ec616063905a87475f08d72daea8caf8ec19 100644 (file)
@@ -39,6 +39,8 @@ import java.io.Serializable;
  */
 public class Hyphen implements Serializable {
 
+    private static final long serialVersionUID = 8989909741110279085L;
+
     /** pre break string */
     public String preBreak;                                     // CSOK: VisibilityModifier
 
index 76e8b6327cc37734dcaaed5db88736803acc4d20..455520b0906284b0274fe961d0fd4e42ebcf42ee 100644 (file)
@@ -43,7 +43,7 @@ import org.xml.sax.InputSource;
 public final class Hyphenator {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(Hyphenator.class);
+    protected static final Log log = LogFactory.getLog(Hyphenator.class);
 
     private static HyphenationTreeCache hTreeCache = null;
 
index a3a82332492315579cd07d4899ac4af5566f7759..206b81f3f628182ef0c7a8fe272aae39116344d0 100644 (file)
@@ -75,6 +75,8 @@ public class TernaryTree implements Cloneable, Serializable {
      * if it ain't broken, don't fix it.
      */
 
+    private static final long serialVersionUID = 3175412271203716160L;
+
     /**
      * Pointer to low branch and to rest of the key when it is
      * stored directly in this node, we don't have unions in java!
index cc18c5b36f7ead05f4e849db2fa09818bc15bb9a..76616bfb542923d0b4c3b5147cd418d8ec88b807 100644 (file)
@@ -40,7 +40,7 @@ import org.apache.fop.util.ListUtil;
 public abstract class AbstractBreaker {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(AbstractBreaker.class);
+    protected static final Log log = LogFactory.getLog(AbstractBreaker.class);
 
     /**
      * A page break position.
@@ -93,6 +93,8 @@ public abstract class AbstractBreaker {
      */
     public class BlockSequence extends BlockKnuthSequence {
 
+        private static final long serialVersionUID = -5348831120146774118L;
+
         /** Number of elements to ignore at the beginning of the list. */
         int ignoreAtStart = 0;                                  // CSOK: VisibilityModifier
         /** Number of elements to ignore at the end of the list. */
index fac5bf0757b673a9d845769bfc925e69c7700349..0be0c9437acb5a84019bc4940a0ae15fde3a973f 100644 (file)
@@ -27,6 +27,8 @@ import java.util.List;
  */
 public class BlockKnuthSequence extends KnuthSequence {
 
+    private static final long serialVersionUID = 1648962416582509095L;
+
     private boolean isClosed = false;
 
     /**
index 15f8ffa8dba23cef4b95f248d4083919767a8e3e..7ac23f74fe147ea3f961bb9ec9b62740cc5fed6d 100644 (file)
@@ -42,7 +42,7 @@ import org.apache.fop.fo.Constants;
 public abstract class BreakingAlgorithm {
 
     /** the logger for the class */
-    protected static Log log = LogFactory.getLog(BreakingAlgorithm.class);
+    protected static final Log log = LogFactory.getLog(BreakingAlgorithm.class);
 
     /** Maximum adjustment ration */
     protected static final int INFINITE_RATIO = 1000;
index 63a2f7c5e2b95bdcb1775b5a2ca4623cb6a73e1a..81df3c992c4a775416e78205b64b3c7055e64c41 100644 (file)
@@ -32,6 +32,8 @@ import org.apache.fop.layoutmgr.inline.KnuthInlineBox;
  */
 public class InlineKnuthSequence extends KnuthSequence  {
 
+    private static final long serialVersionUID = 1354774188859946549L;
+
     private boolean isClosed = false;
 
     /**
index d1b16a52308469ddce015ad3bf4da4e19a253ff0..0667b9355dcc3ebc98ff091362da4b783a8694f9 100644 (file)
@@ -30,7 +30,7 @@ import org.apache.fop.traits.MinOptMax;
 public abstract class UnresolvedListElementWithLength extends UnresolvedListElement {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(UnresolvedListElementWithLength.class);
+    protected static final Log log = LogFactory.getLog(UnresolvedListElementWithLength.class);
 
     private MinOptMax length;
     private boolean conditional;
index 338a58d9c7a48c836680a701fa5b374b00ab788d..73235d2d1282a4b4aceb90bd7e649020ff2c441d 100644 (file)
@@ -37,7 +37,7 @@ import org.apache.fop.fo.properties.LengthRangeProperty;
 public class ImageLayout implements Constants {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(ImageLayout.class);
+    protected static final Log log = LogFactory.getLog(ImageLayout.class);
 
     //Input
     private GraphicsProperties props;
index df7371fe5d8645ccbefab349c8490e8fe4cf7b6f..c4a76f2f5998e5c6cc29c80842a3359734b14b7e 100644 (file)
@@ -56,7 +56,7 @@ public abstract class LeafNodeLayoutManager extends AbstractLayoutManager
     /**
      * logging instance
      */
-    protected static Log log = LogFactory.getLog(LeafNodeLayoutManager.class);
+    protected static final Log log = LogFactory.getLog(LeafNodeLayoutManager.class);
 
     /**
      * The inline area that this leafnode will add.
index 3a03257f8ce5cdf945b9d2189d292bd8c1dbf427..661f6ec80cbdecbd9b6cb1e556cf6fc32384f065 100644 (file)
@@ -184,6 +184,9 @@ public class LineLayoutManager extends InlineStackingLayoutManager
 
     // this class represents a paragraph
     private class Paragraph extends InlineKnuthSequence {
+
+        private static final long serialVersionUID = 5862072380375189105L;
+
         /** Number of elements to ignore at the beginning of the list. */
         private int ignoreAtStart = 0;
         /** Number of elements to ignore at the end of the list. */
index 21ff823945141e748c4ef9bb34bec9f44789c1f3..4bb9a7d34f2fad71b4eb0342d293c4f8f22beda6 100644 (file)
@@ -42,7 +42,7 @@ import org.apache.commons.logging.LogFactory;
 public abstract class PDFObject implements PDFWritable {
 
     /** logger for all PDFObjects (and descendants) */
-    protected static Log log = LogFactory.getLog(PDFObject.class.getName());
+    protected static final Log log = LogFactory.getLog(PDFObject.class.getName());
 
     /**
      * the object's number
index 096007b980ea68ffb07c59447cf0db6bfea925ba..874bb4d1b877bfe36786b494c4f0ab7d76021dc4 100644 (file)
@@ -30,7 +30,7 @@ import org.apache.fop.apps.FOUserAgent;
  */
 public abstract class AbstractConfigurator {
     /** logger instance */
-    protected static Log log = LogFactory.getLog(AbstractConfigurator.class);
+    protected static final Log log = LogFactory.getLog(AbstractConfigurator.class);
 
     private static final String MIME = "mime";
     
index 392eb875eecdf509dbfe4af13f1c70e6f1303c4e..f2c86b35f3b1f50d09260bde4d01061d0d465bad 100644 (file)
@@ -178,7 +178,7 @@ public abstract class AbstractGenericSVGHandler implements XMLHandler, RendererC
     }
 
     /** {@inheritDoc} */
-    public String getNamespace() {
+    public String getNameSpace() {
         return SVGDOMImplementation.SVG_NAMESPACE_URI;
     }
 
index 72be1b0b8b15fada8a05c99cb9346b4be5738c59..d70d8113f6c8b638587376fb0ff0b6a100046e91 100644 (file)
@@ -79,7 +79,7 @@ public abstract class AbstractRenderer
          implements Renderer, Constants {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog("org.apache.fop.render");
+    protected static final Log log = LogFactory.getLog("org.apache.fop.render");
 
     /**
      * user agent
index dd7a0506727540ed0b808eb1e0f75fefa67884f3..8f94684b2028a87083a1874dc694985e51f74c89 100644 (file)
@@ -48,7 +48,7 @@ public class PrintRendererConfigurator extends AbstractRendererConfigurator
             implements RendererConfigurator, IFDocumentHandlerConfigurator {
 
     /** logger instance */
-    protected static Log log = LogFactory.getLog(PrintRendererConfigurator.class);
+    protected static final Log log = LogFactory.getLog(PrintRendererConfigurator.class);
 
     /**
      * Default constructor
index f8152fcbf9316dafff76714ae37109bc967ef75e..1060bec511f707cdadfc6fe5dd54c86df8fbb0b8 100644 (file)
@@ -60,6 +60,6 @@ public interface XMLHandler {
      * @return the XML namespace for the XML dialect this XMLHandler supports,
      * null if all XML content is handled by this instance.
      */
-    String getNamespace();
+    String getNameSpace();
 }
 
index fb24205723d81522436ddc4f20c4bfbea113dfa9..b93d4003a45af3c2db5e6f3388ad8f8bc6b8da40 100644 (file)
@@ -32,7 +32,7 @@ import org.apache.fop.apps.FOUserAgent;
 public class XMLHandlerConfigurator extends AbstractRendererConfigurator {
 
     /** logger instance */
-    protected static Log log = LogFactory.getLog(XMLHandlerConfigurator.class);
+    protected static final Log log = LogFactory.getLog(XMLHandlerConfigurator.class);
 
     /**
      * Default constructor
index 81d9fcc93a3cc0227f76a2e041871412ec192889..84177df45c18f2599ec4db21939dcf6cda64edf7 100644 (file)
@@ -85,7 +85,7 @@ public class XMLHandlerRegistry {
      * @param handler the XMLHandler instance
      */
     public void addXMLHandler(XMLHandler handler) {
-        String ns = handler.getNamespace();
+        String ns = handler.getNameSpace();
         if (ns == null) {
             setDefaultXMLHandler(handler);
         } else {
index 2687d9071a0b4f4d6a279945046d57e050cac2df..cffcf8a25561b94499802c4a8dfe8c215df90cf5 100644 (file)
@@ -124,7 +124,7 @@ public class AFPRendererImageInfo {
      *
      * @return the uri
      */
-    public String getURI() {
+    public String getUri() {
         return this.uri;
     }
 
index f2e77ec3325ee183bf91e83745cebc398de0eedf..d916738eb3090d166eb10cdea1db62d834b597ce 100644 (file)
@@ -41,7 +41,7 @@ public class AFPExtensionHandler extends DefaultHandler
             implements ContentHandlerFactory.ObjectSource {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(AFPExtensionHandler.class);
+    protected static final Log log = LogFactory.getLog(AFPExtensionHandler.class);
 
     private StringBuffer content = new StringBuffer();
     private Attributes lastAttributes;
index 045ce7fe442c34f6559b06b8679d6e30bbacc605..78f09b537de70149c0d9151f34315424187b14a3 100644 (file)
@@ -27,6 +27,8 @@ import java.util.EventObject;
  */
 public class PageChangeEvent extends EventObject {
 
+    private static final long serialVersionUID = -5969283475959932887L;
+
     private int oldPage;
     private int newPage;
 
index 7dd3bcd952f99f3aa0727632f31b1bfa7197ff52..5a9cfa21b2cf126cf5c10ed8dd09ed13108f61e2 100644 (file)
@@ -31,6 +31,7 @@ import java.awt.image.SinglePixelPackedSampleModel;
 import java.io.IOException;
 import java.io.OutputStream;
 import java.util.Iterator;
+import java.util.NoSuchElementException;
 
 import org.apache.commons.logging.Log;
 
@@ -133,7 +134,11 @@ public class TIFFRenderer extends Java2DRenderer implements TIFFConstants {
                 multiWriter.close();
             }
         } else {
-            writer.writeImage((RenderedImage) pageImagesItr.next(), outputStream, writerParams);
+            RenderedImage renderedImage = null;
+            if (pageImagesItr.hasNext()) {
+                renderedImage = (RenderedImage) pageImagesItr.next();
+            }
+            writer.writeImage(renderedImage, outputStream, writerParams);
             if (pageImagesItr.hasNext()) {
                 BitmapRendererEventProducer eventProducer
                     = BitmapRendererEventProducer.Provider.get(
@@ -186,8 +191,7 @@ public class TIFFRenderer extends Java2DRenderer implements TIFFConstants {
             try {
                 pageImage = getPageImage(current++);
             } catch (FOPException e) {
-                log.error(e);
-                return null;
+                throw new NoSuchElementException(e.getMessage());
             }
 
             if (COMPRESSION_CCITT_T4.equalsIgnoreCase(writerParams.getCompressionMethod())
index da5d8a623ec8127c39358700fc7c497bfd6bb20a..645d1ebc6a36626a264d2a341558ebddd5bb92ce 100644 (file)
@@ -71,7 +71,7 @@ import org.apache.fop.util.XMLUtil;
 public class IFParser implements IFConstants {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(IFParser.class);
+    protected static final Log log = LogFactory.getLog(IFParser.class);
 
     private static SAXTransformerFactory tFactory
         = (SAXTransformerFactory)SAXTransformerFactory.newInstance();
index bdeb1e9a1a7667e202e35d98840591de1d5406a1..046d1703b4c36030be5d0c7957fe466cd388a5a7 100644 (file)
@@ -110,7 +110,7 @@ public class IFRenderer extends AbstractPathOrientedRenderer {
     //if optimizations can be done to avoid int->float->int conversions.
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(IFRenderer.class);
+    protected static final Log log = LogFactory.getLog(IFRenderer.class);
 
     /** XML MIME type */
     public static final String IF_MIME_TYPE = MimeConstants.MIME_FOP_IF;
index edeef976655d8de5d4fc8f9873c516ec3665d3cd..dcc0d3523683c581cbac90d8382d1d9e440ab19c 100644 (file)
@@ -668,8 +668,8 @@ public class IFSerializer extends AbstractXMLWritingIFDocumentHandler
             throw new NullPointerException("action must not be null");
         }
         if (!action.isComplete()) {
-            assert action.hasID();
-            incompleteActions.put(action.getID(), action);
+            assert action.hasId();
+            incompleteActions.put(action.getId(), action);
         }
     }
 
@@ -744,8 +744,8 @@ public class IFSerializer extends AbstractXMLWritingIFDocumentHandler
     /** {@inheritDoc} */
     public void addResolvedAction(AbstractAction action) throws IFException {
         assert action.isComplete();
-        assert action.hasID();
-        AbstractAction noted = (AbstractAction)incompleteActions.remove(action.getID());
+        assert action.hasId();
+        AbstractAction noted = (AbstractAction)incompleteActions.remove(action.getId());
         if (noted != null) {
             completeActions.add(action);
         } else {
index 340b2e0681348f642deabe37cebea06bf67abbaf..8d8a30cf615d84cb3dbe75b71f94bcd799fcfc1f 100644 (file)
@@ -33,7 +33,7 @@ public abstract class AbstractAction implements XMLizable {
      * Sets an ID to make the action referencable.
      * @param id the ID
      */
-    public void setID(String id) {
+    public void setId(String id) {
         this.id = id;
     }
 
@@ -41,7 +41,7 @@ public abstract class AbstractAction implements XMLizable {
      * Returns an optional ID for this action.
      * @return the ID or null
      */
-    public String getID() {
+    public String getId() {
         return this.id;
     }
 
@@ -65,7 +65,7 @@ public abstract class AbstractAction implements XMLizable {
      * Indicates whether the action has an ID and is therefore referencable.
      * @return true if the action has an ID
      */
-    public boolean hasID() {
+    public boolean hasId() {
         return this.id != null;
     }
 
@@ -90,7 +90,7 @@ public abstract class AbstractAction implements XMLizable {
      * Returns a string that is used to prefix a generated ID to make it unique.
      * @return the prefix string
      */
-    public String getIDPrefix() {
+    public String getIdPrefix() {
         return null;
     }
 
index b980f1051389c523b71e4d9cdef4f87119110f8c..719d2225bfaea3b8076d4c00c5f9ba0af71ec995 100644 (file)
@@ -39,7 +39,7 @@ public class ActionSet {
      */
     public synchronized String generateNewID(AbstractAction action) {
         this.lastGeneratedID++;
-        String prefix = action.getIDPrefix();
+        String prefix = action.getIdPrefix();
         if (prefix == null) {
             throw new IllegalArgumentException("Action class is not compatible");
         }
@@ -62,12 +62,12 @@ public class ActionSet {
      * @return the action instance that should be used in place of the given one
      */
     public AbstractAction put(AbstractAction action) {
-        if (!action.hasID()) {
-            action.setID(generateNewID(action));
+        if (!action.hasId()) {
+            action.setId(generateNewID(action));
         }
         AbstractAction effAction = normalize(action);
         if (effAction == action) {
-            this.actionRegistry.put(action.getID(), action);
+            this.actionRegistry.put(action.getId(), action);
         }
         return effAction;
     }
index 5ca480f4a74390a4c734b1eb12a1f4d7b9035fcc..1d5bc23fe3feb26eb2e8e9f2fad1890058b2b1ec 100644 (file)
@@ -41,7 +41,7 @@ public class DocumentNavigationHandler extends DefaultHandler
         implements DocumentNavigationExtensionConstants {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(DocumentNavigationHandler.class);
+    protected static final Log log = LogFactory.getLog(DocumentNavigationHandler.class);
 
     private StringBuffer content = new StringBuffer();
     private Stack objectStack = new Stack();
@@ -132,7 +132,7 @@ public class DocumentNavigationHandler extends DefaultHandler
                 boolean newWindow = "new".equals(showDestination);
                 URIAction action = new URIAction(gotoURI, newWindow);
                 if (id != null) {
-                    action.setID(id);
+                    action.setId(id);
                 }
                 if (structurePointer != null) {
                     action.setStructurePointer(structurePointer);
index a2b4f31b61ee40a43ac8a40824eaedee8e146c8d..d905c780039349c1f2eccee59f805ed185517e25 100644 (file)
@@ -52,7 +52,7 @@ public class GoToXYAction extends AbstractAction implements DocumentNavigationEx
      *                  or null, if the position isn't known, yet
      */
     public GoToXYAction(String id, int pageIndex, Point targetLocation) {
-        setID(id);
+        setId(id);
         if (pageIndex < 0 && targetLocation != null) {
             throw new IllegalArgumentException(
                     "Page index may not be null if target location is known!");
@@ -144,7 +144,7 @@ public class GoToXYAction extends AbstractAction implements DocumentNavigationEx
         AttributesImpl atts = new AttributesImpl();
         if (this.isCompleteExceptTargetLocation()) {
             final Point reportedTargetLocation = this.getTargetLocation();
-            atts.addAttribute(null, "id", "id", XMLUtil.CDATA, getID());
+            atts.addAttribute(null, "id", "id", XMLUtil.CDATA, getId());
             atts.addAttribute(null, "page-index", "page-index",
                     XMLUtil.CDATA, Integer.toString(pageIndex));
             atts.addAttribute(null, "x", "x", XMLUtil.CDATA, 
@@ -152,7 +152,7 @@ public class GoToXYAction extends AbstractAction implements DocumentNavigationEx
             atts.addAttribute(null, "y", "y", XMLUtil.CDATA, 
                     Integer.toString(reportedTargetLocation.y));
         } else {
-            atts.addAttribute(null, "idref", "idref", XMLUtil.CDATA, getID());
+            atts.addAttribute(null, "idref", "idref", XMLUtil.CDATA, getId());
         }
         handler.startElement(GOTO_XY.getNamespaceURI(),
                 GOTO_XY.getLocalName(), GOTO_XY.getQName(), atts);
@@ -162,7 +162,7 @@ public class GoToXYAction extends AbstractAction implements DocumentNavigationEx
 
     /** {@inheritDoc} */
     public String toString() {
-        return "GoToXY: ID=" + getID()
+        return "GoToXY: ID=" + getId()
             + ", page=" + getPageIndex()
             + ", loc=" + getTargetLocation() + ", "
             + (isComplete() ? "complete" : "INCOMPLETE");
index 22e7a270c1e89b1117acfe291251f17b24b9d62d..4251619ff7f228e42229e64c7bb9b74616e6c557 100644 (file)
@@ -82,15 +82,15 @@ public class URIAction extends AbstractAction implements DocumentNavigationExten
     }
 
     /** {@inheritDoc} */
-    public String getIDPrefix() {
+    public String getIdPrefix() {
         return "fop-" + GOTO_URI.getLocalName();
     }
 
     /** {@inheritDoc} */
     public void toSAX(ContentHandler handler) throws SAXException {
         AttributesImpl atts = new AttributesImpl();
-        if (hasID()) {
-            atts.addAttribute(null, "id", "id", XMLUtil.CDATA, getID());
+        if (hasId()) {
+            atts.addAttribute(null, "id", "id", XMLUtil.CDATA, getId());
         }
         atts.addAttribute(null, "uri", "uri", XMLUtil.CDATA, getURI());
         if (isNewWindow()) {
index c215243a7d437a1d3647a34c9bb67cdab9ece7c2..4a51cb176ea045cdb43cbbfbb7a384e1af5bd769 100644 (file)
@@ -129,7 +129,7 @@ public class PDFDocumentNavigationHandler implements IFDocumentNavigationHandler
     /** {@inheritDoc} */
     public void addResolvedAction(AbstractAction action) throws IFException {
         assert action.isComplete();
-        PDFAction pdfAction = (PDFAction)this.incompleteActions.remove(action.getID());
+        PDFAction pdfAction = (PDFAction)this.incompleteActions.remove(action.getId());
         if (pdfAction == null) {
             getAction(action);
         } else if (pdfAction instanceof PDFGoTo) {
@@ -145,11 +145,11 @@ public class PDFDocumentNavigationHandler implements IFDocumentNavigationHandler
         if (action == null) {
             return null;
         }
-        PDFAction pdfAction = (PDFAction)this.completeActions.get(action.getID());
+        PDFAction pdfAction = (PDFAction)this.completeActions.get(action.getId());
         if (pdfAction != null) {
             return pdfAction;
         } else if (action instanceof GoToXYAction) {
-            pdfAction = (PDFAction) incompleteActions.get(action.getID());
+            pdfAction = (PDFAction) incompleteActions.get(action.getId());
             if (pdfAction != null) {
                 return pdfAction;
             } else {
@@ -159,7 +159,7 @@ public class PDFDocumentNavigationHandler implements IFDocumentNavigationHandler
                 if (action.isComplete()) {
                     updateTargetLocation(pdfGoTo, a);
                 } else {
-                    this.incompleteActions.put(action.getID(), pdfGoTo);
+                    this.incompleteActions.put(action.getId(), pdfGoTo);
                 }
                 return pdfGoTo;
             }
@@ -173,7 +173,7 @@ public class PDFDocumentNavigationHandler implements IFDocumentNavigationHandler
                 //Some PDF actions are pooled
                 getPDFDoc().registerObject(pdfAction);
             }
-            this.completeActions.put(action.getID(), pdfAction);
+            this.completeActions.put(action.getId(), pdfAction);
             return pdfAction;
         } else {
             throw new UnsupportedOperationException("Unsupported action type: "
@@ -195,7 +195,7 @@ public class PDFDocumentNavigationHandler implements IFDocumentNavigationHandler
 
         //Queue this object now that it's complete
         getPDFDoc().addObject(pdfGoTo);
-        this.completeActions.put(action.getID(), pdfGoTo);
+        this.completeActions.put(action.getId(), pdfGoTo);
     }
 
 }
index 04c95545c2bc1ff7c5cc917982fcbc9527c89747..e7e863a23855bc09f759537689bd2b13c6f0ed19 100644 (file)
@@ -37,7 +37,7 @@ public class PDFExtensionHandler extends DefaultHandler
             implements ContentHandlerFactory.ObjectSource {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(PDFExtensionHandler.class);
+    protected static final Log log = LogFactory.getLog(PDFExtensionHandler.class);
 
     private Attributes lastAttributes;
 
index 8acc390d04ce77fec523afe7aa4127f3dcc34393..0d09750454234509e9827d71f68d5ba8b00d64df 100644 (file)
@@ -54,7 +54,7 @@ import org.apache.fop.fonts.Typeface;
 public class PSFontUtils extends org.apache.xmlgraphics.ps.PSFontUtils {
 
     /** logging instance */
-    protected static Log log = LogFactory.getLog(PSFontUtils.class);
+    protected static final Log log = LogFactory.getLog(PSFontUtils.class);
 
     /**
      * Generates the PostScript code for the font dictionary. This method should only be
index 11c3205e6ffefb5ca40fe280e994c9c6d2f04fd8..4c40c6f926a26355531c7d4d85dba352b7fe2b6e 100644 (file)
@@ -53,7 +53,7 @@ public class PSImageFormResource extends PSResource {
      * Returns the image URI.
      * @return the image URI
      */
-    public String getImageURI() {
+    public String getImageUri() {
         return this.uri;
     }
 
index 502242c17b7afbcbfd9d1e3ca18c086d9c267e4c..06d6b9034fdc1fcdecd40d7d3a2ccd064a8e12b0 100644 (file)
@@ -281,7 +281,7 @@ public class ResourceHandler implements DSCParserConstants, PSSupportedFlavors {
 
     private void generateFormForImage(PSGenerator gen, PSImageFormResource form)
                 throws IOException {
-        final String uri = form.getImageURI();
+        final String uri = form.getImageUri();
 
         ImageManager manager = userAgent.getFactory().getImageManager();
         ImageInfo info = null;
index bf970d0a6134851ba3c0cd58c25974d6889031d7..d499ef6ab6834e3f8fae4f61ffc52a061e53e62b 100644 (file)
@@ -37,7 +37,7 @@ public class PSExtensionHandler extends DefaultHandler
             implements ContentHandlerFactory.ObjectSource {
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(PSExtensionHandler.class);
+    protected static final Log log = LogFactory.getLog(PSExtensionHandler.class);
 
     private StringBuffer content = new StringBuffer();
     private Attributes lastAttributes;
index d377f740d2fcac3c9d726a5dde42315670f29c42..29bbecded8ac231e8d2511fd871c296a5daf2435 100644 (file)
@@ -149,7 +149,7 @@ final class TextAttributesConverter {
         attrFont(fobj.getCommonFont(), attrib);
         attrFontColor(fobj.getColor(), attrib);
         attrTextDecoration(fobj.getTextDecoration(), attrib);
-        attrBaseLineShift(fobj.getBaseLineShift(), attrib);
+        attrBaseLineShift(fobj.getBaselineShift(), attrib);
         return attrib;
     }
 
index f231a1f06f2e4bf78de4912a941523b4f736a538..146a8c19ced1381fdbf1ca28c5fc94d22087b0e3 100644 (file)
@@ -37,23 +37,10 @@ public final class ParagraphKeeptogetherContext {
 
     private static int paraKeepTogetherOpen = 0;
     private static boolean paraResetProperties = false;
-    private static ParagraphKeeptogetherContext instance = null;
 
     private ParagraphKeeptogetherContext() {
     }
 
-    /**
-     * Singelton.
-     *
-     * @return The instance of ParagraphKeeptogetherContext
-     */
-    public static ParagraphKeeptogetherContext getInstance() {
-        if (instance == null) {
-            instance = new ParagraphKeeptogetherContext();
-        }
-        return instance;
-    }
-
     /**
      *  @return the level of current "keep whith next" paragraph
      */
index d7b77f6334ecdaea6ef9aa58ae3b0676b014b55d..b9e53d42f0902c5e67ac0eac7ba123e4b0a42090 100644 (file)
@@ -39,14 +39,12 @@ public class RtfJforCmd extends RtfContainer {
     private static final String PARA_KEEP_OFF = "para-keep:off";
 
     private final RtfAttributes attrib;
-    private ParagraphKeeptogetherContext paragraphKeeptogetherContext;
 
 
 
     RtfJforCmd(RtfContainer parent, Writer w, RtfAttributes attrs) throws IOException {
         super((RtfContainer)parent, w);
         attrib = attrs;
-        paragraphKeeptogetherContext = ParagraphKeeptogetherContext.getInstance();
     }
 
 
index 128c9a3dfe2a25a0a43b22fe3656b94ebaaa4d56..927159ffc78480e2a1594472497b358ca3c7adf4 100644 (file)
@@ -54,7 +54,7 @@ public class RtfTextrun extends RtfContainer {
     /**
      * logging instance
      */
-    protected static Log log = LogFactory.getLog(RtfTextrun.class);
+    protected static final Log log = LogFactory.getLog(RtfTextrun.class);
 
     /** Manager for handling space-* property. */
     private RtfSpaceManager rtfSpaceManager = new RtfSpaceManager();
@@ -271,7 +271,6 @@ public class RtfTextrun extends RtfContainer {
         //add RtfSpaceSplitter to inherit accumulated space
         rtfSpaceManager.pushRtfSpaceSplitter(attrs);
         rtfSpaceManager.setCandidate(attrs);
-        RtfString r = new RtfString(this, writer, s);
         rtfSpaceManager.popRtfSpaceSplitter();
     }
 
@@ -301,7 +300,7 @@ public class RtfTextrun extends RtfContainer {
       ListIterator lit = children.listIterator(children.size());
       while (lit.hasPrevious()
               && (lit.previous() instanceof RtfCloseGroupMark)) {
-          tmp.push(new Integer(((RtfCloseGroupMark)lit.next()).getBreakType()));
+          tmp.push(Integer.valueOf(((RtfCloseGroupMark)lit.next()).getBreakType()));
           lit.remove();
           deletedCloseGroupCount++;
       }
index d1cf1cf9a1dd2b93712c694c4d954e8db8e5782e..f181e19110d632e76a5cd327aab5592242117c5a 100644 (file)
@@ -48,7 +48,7 @@ public class XMLXMLHandler implements XMLHandler {
     }
 
     /** {@inheritDoc} */
-    public String getNamespace() {
+    public String getNameSpace() {
         return null; //Handle all XML content
     }
 
index d446dff54b93c897fedf5f2cc7421bd52780cd6d..5c243ecaecbc40a02a0bf933b93b9a198c15219a 100644 (file)
@@ -61,6 +61,8 @@ import org.apache.fop.apps.MimeConstants;
  */
 public class FopPrintServlet extends FopServlet {
 
+    private static final long serialVersionUID = 1645706757391617935L;
+
     /**
      * {@inheritDoc}
      */
index 304cf21b9190f0daebf1694291ad724e5aaa2433..7eb8a2f8955a4f891ac9a18e78ef098e62d285d9 100644 (file)
@@ -70,6 +70,8 @@ import org.apache.fop.apps.MimeConstants;
  */
 public class FopServlet extends HttpServlet {
 
+    private static final long serialVersionUID = -908918093488215264L;
+
     /** Name of the parameter used for the XSL-FO file */
     protected static final String FO_REQUEST_PARAM = "fo";
     /** Name of the parameter used for the XML file */
index 0c6e09bd0e1ae129264535d8a67b181b70f0a119..ba03ee567fbecf834903c72d43116c988082d30c 100644 (file)
@@ -56,7 +56,7 @@ public class TestConverter {
     private String outputFormat = MimeConstants.MIME_FOP_AREA_TREE;
     private File destdir;
     private File compare = null;
-    private String baseDir = "./";
+    private String basedir = "./";
     private Map differ = new java.util.HashMap();
 
     /**
@@ -95,7 +95,7 @@ public class TestConverter {
             } else if (args[count].equals("-d")) {
                 tc.setDebug(true);
             } else if (args[count].equals("-b")) {
-                tc.setBaseDir(args[++count]);
+                tc.setBasedir(args[++count]);
             } else if (args[count].equals("-results")) {
                 results = args[++count];
             } else {
@@ -138,8 +138,8 @@ public class TestConverter {
      * Sets the base directory.
      * @param str base directory
      */
-    public void setBaseDir(String str) {
-        baseDir = str;
+    public void setBasedir(String str) {
+        basedir = str;
     }
 
     /**
@@ -167,11 +167,11 @@ public class TestConverter {
         logger.debug("running tests in file:" + fname);
         try {
             if (compDir != null) {
-                compare = new File(baseDir + "/" + compDir);
+                compare = new File(basedir + "/" + compDir);
             }
-            destdir = new File(baseDir + "/" + dest);
+            destdir = new File(basedir + "/" + dest);
             destdir.mkdirs();
-            File f = new File(baseDir + "/" + fname);
+            File f = new File(basedir + "/" + fname);
             DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
             DocumentBuilder db = factory.newDocumentBuilder();
             Document doc = db.parse(f);
@@ -271,7 +271,7 @@ public class TestConverter {
             res = resNode.getNodeValue();
         }
         try {
-            File xmlFile = new File(baseDir + "/" + xml);
+            File xmlFile = new File(basedir + "/" + xml);
             String baseURL = null;
             try {
                 baseURL = xmlFile.getParentFile().toURI().toURL().toExternalForm();
@@ -284,7 +284,7 @@ public class TestConverter {
                 inputHandler = new InputHandler(xmlFile);
             } else {
                 inputHandler = new InputHandler(xmlFile,
-                                                new File(baseDir + "/"
+                                                new File(basedir + "/"
                                                          + xsl), null);
             }
 
index 58dd1fb5b20eaa9794307703e58ffc740e0e9a0d..3aeac1215c5f7df22f634dba71f18c95a733acdc 100644 (file)
@@ -85,7 +85,7 @@ public class Fop extends Task {
      * Sets the filename for the userconfig.xml.
      * @param userConfig Configuration to use
      */
-    public void setUserconfig(File userConfig) {
+    public void setUserConfig(File userConfig) {
         this.userConfig = userConfig;
     }
 
@@ -93,7 +93,7 @@ public class Fop extends Task {
      * Returns the file for the userconfig.xml.
      * @return the userconfig.xml file
      */
-    public File getUserconfig() {
+    public File getUserConfig() {
         return this.userConfig;
     }
 
@@ -101,7 +101,7 @@ public class Fop extends Task {
      * Sets the input XSL-FO file.
      * @param foFile input XSL-FO file
      */
-    public void setFofile(File foFile) {
+    public void setFoFile(File foFile) {
         this.foFile = foFile;
     }
 
@@ -109,7 +109,7 @@ public class Fop extends Task {
      * Gets the input XSL-FO file.
      * @return input XSL-FO file
      */
-    public File getFofile() {
+    public File getFoFile() {
         return foFile;
     }
 
@@ -267,7 +267,7 @@ public class Fop extends Task {
      * default is false.
      * @param throwExceptions true if exceptions should be thrown
      */
-    public void setThrowexceptions(boolean throwExceptions) {
+    public void setThrowExceptions(boolean throwExceptions) {
         this.throwExceptions = throwExceptions;
     }
 
@@ -275,7 +275,7 @@ public class Fop extends Task {
      * Gets the throw exceptions attribute
      * @return the throw exceptions attribute
      */
-    public boolean getThrowexceptions() {
+    public boolean getThrowExceptions() {
         return this.throwExceptions;
     }
 
@@ -408,8 +408,8 @@ class FOPTaskStarter {
 
     FOPTaskStarter(Fop task) throws SAXException, IOException {
         this.task = task;
-        if (task.getUserconfig() != null) {
-            fopFactory.setUserConfig(task.getUserconfig());
+        if (task.getUserConfig() != null) {
+            fopFactory.setUserConfig(task.getUserConfig());
         }
     }
 
@@ -497,8 +497,8 @@ class FOPTaskStarter {
             }
         } else {
             try {
-                if (task.getFofile() != null) {
-                    this.baseURL =  task.getFofile().getParentFile().toURI().toURL().
+                if (task.getFoFile() != null) {
+                    this.baseURL =  task.getFoFile().getParentFile().toURI().toURL().
                                       toExternalForm();
                 }
             } catch (MalformedURLException mfue) {
@@ -517,8 +517,8 @@ class FOPTaskStarter {
         int skippedcount = 0;
 
         // deal with single source file
-        if (task.getFofile() != null) {
-            if (task.getFofile().exists()) {
+        if (task.getFoFile() != null) {
+            if (task.getFoFile().exists()) {
                 File outf = task.getOutfile();
                 if (outf == null) {
                     throw new BuildException("outfile is required when fofile is used");
@@ -530,11 +530,11 @@ class FOPTaskStarter {
                 // OR output file doesn't exist OR
                 // output file is older than input file
                 if (task.getForce() || !outf.exists()
-                    || (task.getFofile().lastModified() > outf.lastModified() )) {
-                    render(task.getFofile(), outf, outputFormat);
+                    || (task.getFoFile().lastModified() > outf.lastModified() )) {
+                    render(task.getFoFile(), outf, outputFormat);
                     actioncount++;
                 } else if (outf.exists()
-                        && (task.getFofile().lastModified() <= outf.lastModified() )) {
+                        && (task.getFoFile().lastModified() <= outf.lastModified() )) {
                     skippedcount++;
                 }
             }
@@ -653,7 +653,7 @@ class FOPTaskStarter {
             inputHandler.renderTo(userAgent, outputFormat, out);
             success = true;
         } catch (Exception ex) {
-            if (task.getThrowexceptions()) {
+            if (task.getThrowExceptions()) {
                 throw new BuildException(ex);
             }
             throw ex;
index 25ac86cedcea65e5a7316f178be2735ef81a22b6..a76c6e8a3f9f8d802002ab468efc25e40a0b2baf 100644 (file)
@@ -34,6 +34,8 @@ import org.apache.fop.util.ColorUtil;
  */
 public class BorderProps implements Serializable {
 
+    private static final long serialVersionUID = -886871454032189183L;
+
     /** Separate border model */
     public static final int SEPARATE = 0;
     /** Collapsing border model, for borders inside a table */
index 8fe6c707f3845c4fd87168c86e858abe488ebf2a..21277d60afaf136c1b413173b7dd8b59fc5b8c3a 100644 (file)
@@ -53,7 +53,7 @@ public final class ColorUtil {
     private static Map colorMap = null;
 
     /** Logger instance */
-    protected static Log log = LogFactory.getLog(ColorUtil.class);
+    protected static final Log log = LogFactory.getLog(ColorUtil.class);
 
     static {
         initializeColorMap();