]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Fixed javadoc errors.
authorJoerg Pietschmann <pietsch@apache.org>
Thu, 8 Sep 2005 22:06:48 +0000 (22:06 +0000)
committerJoerg Pietschmann <pietsch@apache.org>
Thu, 8 Sep 2005 22:06:48 +0000 (22:06 +0000)
Added a few TODOs.

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

43 files changed:
src/java/org/apache/fop/apps/FOPException.java
src/java/org/apache/fop/area/BookmarkData.java
src/java/org/apache/fop/area/CTM.java
src/java/org/apache/fop/area/inline/InlineArea.java
src/java/org/apache/fop/datatypes/LengthBase.java
src/java/org/apache/fop/datatypes/Numeric.java
src/java/org/apache/fop/datatypes/SimplePercentBaseContext.java
src/java/org/apache/fop/fo/expr/NumericProperty.java
src/java/org/apache/fop/fo/expr/RelativeNumericProperty.java
src/java/org/apache/fop/fo/pagination/Root.java
src/java/org/apache/fop/fo/properties/BoxPropShorthandParser.java
src/java/org/apache/fop/fo/properties/CharacterProperty.java
src/java/org/apache/fop/fo/properties/ColorTypeProperty.java
src/java/org/apache/fop/fo/properties/CompoundPropertyMaker.java
src/java/org/apache/fop/fo/properties/CondLengthProperty.java
src/java/org/apache/fop/fo/properties/EnumLength.java
src/java/org/apache/fop/fo/properties/EnumProperty.java
src/java/org/apache/fop/fo/properties/FixedLength.java
src/java/org/apache/fop/fo/properties/KeepProperty.java
src/java/org/apache/fop/fo/properties/LengthProperty.java
src/java/org/apache/fop/fo/properties/LengthRangeProperty.java
src/java/org/apache/fop/fo/properties/LineHeightPropertyMaker.java
src/java/org/apache/fop/fo/properties/NumberProperty.java
src/java/org/apache/fop/fo/properties/PercentLength.java
src/java/org/apache/fop/fo/properties/PropertyMaker.java
src/java/org/apache/fop/fo/properties/SpaceProperty.java
src/java/org/apache/fop/fo/properties/StringProperty.java
src/java/org/apache/fop/fo/properties/TableColLength.java
src/java/org/apache/fop/fonts/FontSetup.java
src/java/org/apache/fop/layoutmgr/BlockStackingLayoutManager.java
src/java/org/apache/fop/layoutmgr/KnuthElement.java
src/java/org/apache/fop/layoutmgr/inline/AbstractGraphicsLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/ContentLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/InlineLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/TextLayoutManager.java
src/java/org/apache/fop/layoutmgr/list/ListBlockLayoutManager.java
src/java/org/apache/fop/layoutmgr/table/CollapsingBorderModel.java
src/java/org/apache/fop/layoutmgr/table/TableContentLayoutManager.java
src/java/org/apache/fop/pdf/PDFEncryptionManager.java
src/java/org/apache/fop/render/java2d/Java2DRenderer.java
src/java/org/apache/fop/svg/SVGUserAgent.java
src/java/org/apache/fop/tools/anttasks/FileCompare.java
src/java/org/apache/fop/traits/MinOptMax.java

index 5a496bd67f4174b18f89eeafcb0cbcbc8258bf89..10fe45fbea2f839f60faa81ebdadca1cf5b1d43b 100644 (file)
@@ -84,7 +84,7 @@ public class FOPException extends SAXException {
 
     /**
      * Set a location associated with the exception.
-     * @param location the locator holding the location.
+     * @param locator the locator holding the location.
      */
     public void setLocator(Locator locator) {
         if (locator != null) {
index 92f008d97e4b8ec7eed7bb3898aa3e0e0db8e918..ab022e7d90a814d614932858c8e701f0a6e20acf 100644 (file)
@@ -177,7 +177,7 @@ public class BookmarkData extends AbstractOffDocumentItem implements Resolvable
      * id reference.
      *
      * @see org.apache.fop.area.Resolvable#resolveIDRef(String, List)
-     * @todo check to make sure works when multiple bookmark-items
+     * @todo check to make sure it works if multiple bookmark-items
      * have the same idref
      */
     public void resolveIDRef(String id, List pages) {
index 265121626ca8a9e694232bc93acc11b65d0c7ff4..a4eab7c2ead6b2f5d1ce555ba6f06ca27b517984 100644 (file)
@@ -247,7 +247,7 @@ public class CTM implements Serializable {
     /**
      * Construct a coordinate transformation matrix (CTM).
      * @param absRefOrient absolute reference orientation
-     * @param writingmode the writing mode
+     * @param writingMode the writing mode
      * @param absVPrect absolute viewpoint rectangle
      * @param reldims relative dimensions
      * @return CTM the coordinate transformation matrix (CTM)
index f187a4080ede4643b3d7daa9292c3753557327a1..e448a45165eec564a59d0957ef7d1d49c921d53e 100644 (file)
@@ -131,7 +131,7 @@ public class InlineArea extends Area {
     /**
      * Set the parent for the child area.
      *
-     * @see org.apache.fop.area.inline.Area#addChildArea(Area)
+     * @see org.apache.fop.area.Area#addChildArea(Area)
      */
     public void addChildArea(Area childArea) {
         super.addChildArea(childArea);
index 42960b0c1cbe0bda0f53db69e4697c148e23eb0f..633643ecb5cb5d3da456282a757311d5a607cb0e 100644 (file)
@@ -110,7 +110,9 @@ public class LengthBase implements PercentBase {
         return 1.0;
     }
 
-    /** @see org.apache.fop.datatypes.PercentBase#getBaseLength() */
+    /**
+     * @see org.apache.fop.datatypes.PercentBase#getBaseLength(PercentBaseContext)
+     */
     public int getBaseLength(PercentBaseContext context) throws PropertyException {
         int baseLength = 0;
         if (context != null) {
index c8b9af75c0213ec95c5ec39e0351e93e1ccbcd44..92bc3d10749667221a0c539fde91093fade79b57 100644 (file)
@@ -71,7 +71,7 @@ public interface Numeric {
 
     /**
      * Returns the value of this numeric as an int.
-     * @param conext The context for the length calculation (for percentage based lengths)
+     * @param context the context for the length calculation (for percentage based lengths)
      * @return the value as an integer.
      */
     public int getValue(PercentBaseContext context);
index 1b4bbd8bc7376f562bc129d120953e719d5a705f..8b0688fe16522f71579a73b64f5546d7aa324b63 100644 (file)
@@ -44,7 +44,7 @@ public class SimplePercentBaseContext implements PercentBaseContext {
 
     /**
      * Returns the value for the given lengthBase.
-     * @see org.apache.fop.datatypes.PercentBaseContext#getBaseLength(int, fobj)
+     * @see org.apache.fop.datatypes.PercentBaseContext#getBaseLength(int, FObj)
      */
     public int getBaseLength(int lengthBase, FObj fobj) {
         // if its for us return our value otherwise delegate to parent context
index dd146ce7ff46f983f22bced5dbb4941229be16d6..ba00474483fe4474d5caf0d11bec1c8892f63095 100644 (file)
@@ -61,9 +61,7 @@ public class NumericProperty extends Property implements Numeric, Length {
     }
 
     /**
-     * Return the value.
-     * @param Evaluation context
-     * @see Numeric#getNumericValue(Object)
+     * @see Numeric#getNumericValue(PercentBaseContext)
      */
     public double getNumericValue(PercentBaseContext context) {
         return value;
@@ -92,15 +90,14 @@ public class NumericProperty extends Property implements Numeric, Length {
     }
 
     /**
-     * Return the value of this numeric as a length in millipoints. 
+     * @see org.apache.fop.datatypes.Numeric#getValue()
      */
     public int getValue() {
         return (int) value;
     }
 
     /**
-     * Return the value of this numeric as a length in millipoints. 
-     * @param Evaluation context
+     * @see org.apache.fop.datatypes.Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         return (int) value;
index 1d4a921d3422b9edfb95bd3993d15c99ded33a6f..07a2e24400dc89c9407b0b528890d4a150fb982d 100755 (executable)
@@ -128,16 +128,14 @@ public class RelativeNumericProperty extends Property implements Numeric, Length
 
     /**
      * Return the resolved (calculated) value of the expression. 
-     * @see Numeric#getNumericValue()
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue()
      */
     public double getNumericValue() throws PropertyException {
         return getResolved(null).getNumericValue(null);
     }
 
     /**
-     * Return the value.
-     * @param Evaluation context
-     * @see Numeric#getNumericValue(Object)
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue(PercentBaseContext)
      */
     public double getNumericValue(PercentBaseContext context) throws PropertyException {
         return getResolved(context).getNumericValue(context);
@@ -174,7 +172,7 @@ public class RelativeNumericProperty extends Property implements Numeric, Length
     }
 
     /**
-     * Return a resolved length.
+     * @see org.apache.fop.datatypes.Numeric#getValue()
      */
     public int getValue() {
         try {
@@ -186,8 +184,7 @@ public class RelativeNumericProperty extends Property implements Numeric, Length
     }
 
     /**
-     * Return the value of this numeric as a length in millipoints. 
-     * @param Evaluation context
+     * @see org.apache.fop.datatypes.Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         try {
@@ -199,7 +196,8 @@ public class RelativeNumericProperty extends Property implements Numeric, Length
     }
 
     /**
-     * Return a string represention of the expression. Only used for debugging. 
+     * Return a string represention of the expression. Only used for debugging.
+     * @return the string representation.
      */
     public String toString() {
         switch (operation) {
index 4d27222deaad4c06f7f76fda566894dde257ad39..cbd87c43fda8e170dda2611bb6bfeedc5df0091c 100644 (file)
@@ -236,7 +236,7 @@ public class Root extends FObj {
 
     /**
      * Sets the associated Declarations.
-     * @param Declarations the Declarations to use
+     * @param declarations the Declarations to use
      */
     public void setDeclarations(Declarations declarations) {
         this.declarations = declarations;
index 83c4f46a45778e93d8972d45c3eee844a89b60cc..9d99c0d1173c5879f1b8b50303f9f26343e10ff2 100644 (file)
@@ -28,7 +28,7 @@ import org.apache.fop.fo.expr.PropertyException;
 public class BoxPropShorthandParser extends GenericShorthandParser {
 
     /**
-     * @see org.apache.fop.fo.GenericShorthandParser#GenericShorthandParser()
+     * Default constructor.
      */
     public BoxPropShorthandParser() {
     }
@@ -38,7 +38,7 @@ public class BoxPropShorthandParser extends GenericShorthandParser {
      * Set the given property based on the number of values set.
      * Example: padding, border-width, border-color, border-style, margin
      * @see org.apache.fop.fo.GenericShorthandParser#convertValueForProperty(
-     * int, ListProperty, PropertyMaker, PropertyList)
+     * int, Property, PropertyMaker, PropertyList)
      */
     protected Property convertValueForProperty(int propId,
                                                Property property,
index d5a91268121bd9518daeeceef925ec053c4c82da..bf2e9db00c77a345f01bdcc9b6fef91e9f29388b 100644 (file)
@@ -32,7 +32,7 @@ public class CharacterProperty extends Property {
     public static class Maker extends PropertyMaker {
 
         /**
-         * @param propName name of property for which a Maker should be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 99311cc98d5b5f3b717dccf1e156e3b5f8c4c3c1..6d3fc6b733cf9f827c93e72ed3645f0b09c13296 100644 (file)
@@ -57,7 +57,7 @@ public class ColorTypeProperty extends Property implements ColorType {
     public static class Maker extends PropertyMaker {
 
         /**
-         * @param propName name of property for which a Maker should be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 1f8e5777958d1fc9920744199a16af129a1ca816..7fc2495bfa629be8f58a56c591701aa3d817d314 100644 (file)
@@ -93,7 +93,7 @@ public class CompoundPropertyMaker extends PropertyMaker {
      * of compound property types, such as "space".
      * Overridden by property maker subclasses which handle
      * compound properties.
-     * @param subprop The Constants ID of the component for which a Maker is to
+     * @param subpropId the id of the component for which a Maker is to
      * returned, for example CP_OPTIMUM, if the FO attribute is
      * space.optimum='10pt'.
      * @return the Maker object specified
@@ -139,7 +139,7 @@ public class CompoundPropertyMaker extends PropertyMaker {
      * the default value.
      * @param subpropId  The subproperty id of the property being retrieved.
      *        Is 0 when retriving a base property.
-     * @param propertylist The PropertyList object being built for this FO.
+     * @param propertyList The PropertyList object being built for this FO.
      * @param bTryInherit true if inherited properties should be examined.
      * @param bTryDefault true if the default value should be returned. 
      */
index 0253e8a8152e39819fb75bf3adbfd481066699bc..a5188f2ebeddc23969d130baf06fd9cb7baba5ef 100644 (file)
@@ -39,7 +39,7 @@ public class CondLengthProperty extends Property implements CompoundDatatype {
     public static class Maker extends CompoundPropertyMaker {
 
         /**
-         * @param name of property for which a Maker should be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 04efebaa6bc33e61afb2ba14414a6a72808ff656..910335613315ec60e6835525e3ce3b014b10c3f3 100755 (executable)
@@ -42,8 +42,7 @@ public class EnumLength extends LengthProperty {
     }
 
     /**
-     * Returns the length in 1/1000ths of a point (millipoints)
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getValue()
      */
     public int getValue() {
         log.error("getValue() called on " + enumProperty + " length");
@@ -51,9 +50,7 @@ public class EnumLength extends LengthProperty {
     }
 
     /**
-     * Returns the length in 1/1000ths of a point (millipoints)
-     * @param Evaluation context
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         log.error("getValue() called on " + enumProperty + " length");
@@ -61,8 +58,7 @@ public class EnumLength extends LengthProperty {
     }
 
     /**
-     * Returns the value as numeric.
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue()
      */
     public double getNumericValue() {
         log.error("getNumericValue() called on " + enumProperty + " number");
@@ -70,9 +66,7 @@ public class EnumLength extends LengthProperty {
     }
 
     /**
-     * Returns the value as numeric.
-     * @param context Evaluation context
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue()
      */
     public double getNumericValue(PercentBaseContext context) {
         log.error("getNumericValue() called on " + enumProperty + " number");
index 0e1a90bf2f17dd14300a80c2b33076c772e7f0c4..658e52c0706eed687b33f52b22b502ae500198b3 100644 (file)
@@ -33,7 +33,7 @@ public class EnumProperty extends Property {
     public static class Maker extends PropertyMaker {
 
         /**
-         * @param propName name of property for which a Maker should be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 347051990a445884d9aae47b8ecc80aa317791d5..62c7c1c4dfdfc4cdbcffc4f465de1b4e14756224 100644 (file)
@@ -58,6 +58,7 @@ public class FixedLength extends LengthProperty {
      * @param unit input unit specifier (in, cm, etc.)
      */
     protected void convert(double dvalue, String unit) {
+        // TODO: the whole routine smells fishy.
 
         int assumedResolution = 1;    // points/pixel
 
@@ -71,6 +72,7 @@ public class FixedLength extends LengthProperty {
             // Do nothing.
             // dvalue = dvalue;
         } else if (unit.equals("mpt")) { //mpt is non-standard!!! mpt=millipoints
+            // TODO: this seems to be wrong.
             // Do nothing.
             // dvalue = dvalue;
         } else if (unit.equals("pc")) {
@@ -80,6 +82,7 @@ public class FixedLength extends LengthProperty {
              * dvalue = dvalue * fontsize;
              */
         } else if (unit.equals("px")) {
+            // TODO: get resolution from user agent?
             dvalue = dvalue * assumedResolution;
         } else {
             dvalue = 0;
@@ -93,36 +96,28 @@ public class FixedLength extends LengthProperty {
     }
 
     /**
-     * Returns the length in 1/1000ths of a point (millipoints)
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getValue()
      */
     public int getValue() {
         return millipoints;
     }
 
     /**
-     * Returns the length in 1/1000ths of a point (millipoints)
-     * @param Evaluation context
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         return millipoints;
     }
 
     /**
-     * Returns the value as numeric.
-     * @return the length in millipoints
-     * @see Numeric#getNumericValue()
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue()
      */
     public double getNumericValue() {
         return millipoints;
     }
 
     /**
-     * Return the value of this Numeric.
-     * @param context Evaluation context
-     * @return the length in millipoints
-     * @see Numeric#getNumericValue(Object)
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue(PercentBaseContext)
      */
     public double getNumericValue(PercentBaseContext context) {
         return millipoints;
index e0a92886a6dd8a1d2382346dad004c698b595353..14f0e54a232f200dc2c97a85a2bfd91135ba27a7 100644 (file)
@@ -37,7 +37,7 @@ public class KeepProperty extends Property implements CompoundDatatype {
     public static class Maker extends CompoundPropertyMaker {
 
         /**
-         * @param name name of property for which Maker should be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 78646d87cce8a4b9f3e76797f6d15b16a95794c0..243414605345cde8719c3e4d3067fdb8a9ac476d 100644 (file)
@@ -37,7 +37,7 @@ abstract public class LengthProperty extends Property
         private boolean autoOk = false;
 
         /**
-         * @param name name of property for which Maker should be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 37e40028be299fa75bfd0759de1eedc764a811b1..b11d4b8069129d930d6c687a946bb8d0bf3656f2 100644 (file)
@@ -43,7 +43,7 @@ public class LengthRangeProperty extends Property implements CompoundDatatype {
     public static class Maker extends CompoundPropertyMaker {
 
         /**
-         * @param name name of property for which to create Maker
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 269b0c2e376996485e42212ef57249a2d4232388..b2194dc97682f7e6f6bac3b5b240129e6c886192 100644 (file)
@@ -24,9 +24,9 @@ import org.apache.fop.fo.expr.PropertyException;
 
 /**
  * A maker which calculates the line-height property.
- * This property maker is special because line-height inherit the specified
+ * This property maker is special because line-height inherits the specified
  * value, instead of the computed value.
- * So when a line-height is create based on a attribute, the specified value
+ * So when a line-height is create based on an attribute, the specified value
  * is stored in the property and in compute() the stored specified value of
  * the nearest specified is used to recalculate the line-height.  
  */
@@ -60,7 +60,7 @@ public class LineHeightPropertyMaker extends SpaceProperty.Maker {
     }
 
     /**
-     * @see SpaceProperty#convertProperty(Property, PropertyList, FObj)
+     * @see SpaceProperty.Maker#convertProperty(Property, PropertyList, FObj)
      */
     public Property convertProperty(Property p,
             PropertyList propertyList,
index 268066c570bf8935f4e13952300664acd7ea08b0..c3f3788e0e4a1f14a6373bc29064e01557bb6809 100644 (file)
@@ -35,7 +35,7 @@ public class NumberProperty extends Property implements Numeric {
 
         /**
          * Constructor for NumberProperty.Maker
-         * @param propName the name of the property
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
@@ -109,7 +109,7 @@ public class NumberProperty extends Property implements Numeric {
      * Return the value of this Numeric.
      * @param context Evaluation context
      * @return The value as a double.
-     * @see Numeric#getNumericValue(Object)
+     * @see Numeric#getNumericValue(PercentBaseContext)
      */
     public double getNumericValue(PercentBaseContext context) {
         return getNumericValue();
@@ -123,7 +123,7 @@ public class NumberProperty extends Property implements Numeric {
      * Return the value
      * @param context Evaluation context
      * @return The value as an int.
-     * @see Numeric#getValue(Object)
+     * @see Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         return getValue();
index ccad91d79f60fd4ea0a530075a4d43197f8d6daa..b17785c36ecf5e462e997f9eeff7b20556794cfd 100644 (file)
@@ -79,7 +79,6 @@ public class PercentLength extends LengthProperty {
     }
 
     /**
-     * Return the value of this Numeric.
      * @see org.apache.fop.datatypes.Numeric#getNumericValue()
      */
     public double getNumericValue() {
@@ -87,10 +86,7 @@ public class PercentLength extends LengthProperty {
     }
 
     /**
-     * Return the value of this Numeric.
-     * @param context Evaluation context
-     * @return the length in millipoints
-     * @see Numeric#getNumericValue(Object)
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue(PercentBaseContext)
      */
     public double getNumericValue(PercentBaseContext context) {
         try {
@@ -111,8 +107,7 @@ public class PercentLength extends LengthProperty {
     }
 
     /**
-     * Return the value of this numeric as a length in millipoints. 
-     * @param Evaluation context
+     * @see org.apache.fop.datatypes.Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         return (int) getNumericValue(context);
index 6f0d86bfb6a688387876b49b9de82fac7ef99119..a719d58811d865993ca1682126090c3d10621110 100644 (file)
@@ -163,7 +163,7 @@ public class PropertyMaker implements Cloneable {
 
     /**
      * Set the shorthand datatype parser.
-     * @param subproperty
+     * @param parser the shorthand parser
      */
     public void setDatatypeParser(ShorthandParser parser) {
         datatypeParser = parser;
@@ -197,11 +197,11 @@ public class PropertyMaker implements Cloneable {
     }
 
     /**
-     * Set the byShorthand flag which only is applicable for subproperty 
+     * Set the setByShorthand flag which only is applicable for subproperty 
      * makers. It should be true for the subproperties which must be 
      * assigned a value when the base property is assigned a attribute 
      * value directly.
-     * @param defaultValue
+     * @param setByShorthand
      */
     public void setByShorthand(boolean setByShorthand) {
         this.setByShorthand = setByShorthand;
@@ -273,7 +273,7 @@ public class PropertyMaker implements Cloneable {
      * the default value.
      * @param subpropId  The subproperty id of the property being retrieved.
      *        Is 0 when retriving a base property.
-     * @param propertylist The PropertyList object being built for this FO.
+     * @param propertyList The PropertyList object being built for this FO.
      * @param bTryInherit true if inherited properties should be examined.
      * @param bTryDefault true if the default value should be returned. 
      */
@@ -319,7 +319,7 @@ public class PropertyMaker implements Cloneable {
      * property.
      * @param p A property value for a compound property type such as
      * SpaceProperty.
-     * @param subprop The Constants ID of the component whose value is to be
+     * @param subpropId the id of the component whose value is to be
      * returned.
      * NOTE: this is only to ease porting when calls are made to
      * PropertyList.get() using a component name of a compound property,
@@ -503,8 +503,8 @@ public class PropertyMaker implements Cloneable {
      * initializers "thin", "medium", or "thick". The FOPropertyMapping
      * file specifies a length value equivalent for these keywords,
      * such as "0.5pt" for "thin".
-     * @param value The string value of property attribute.
-     * @return A String containging a parseable equivalent or null if
+     * @param keyword the string value of property attribute.
+     * @return a String containing a parseable equivalent or null if
      * the passed value isn't a keyword initializer for this Property.
      */
     protected String checkValueKeywords(String keyword) {
@@ -514,6 +514,7 @@ public class PropertyMaker implements Cloneable {
                 return value;
             }
         }
+        // TODO: should return null here?
         return keyword;            
     }
 
index 45dce11a966df3f8a23a7fc1a5f5694452877b4f..3d667c0f09cf3ffdf3e957642785dd2a6193bda0 100644 (file)
@@ -38,7 +38,7 @@ public class SpaceProperty extends LengthRangeProperty {
     public static class Maker extends CompoundPropertyMaker {
 
         /**
-         * @param name name of the property whose Maker is to be created
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index 9da2ea4b85250b462562391b4b772661207b2308..04e673e170865c19edcce08defbdf4affea4a8c1 100644 (file)
@@ -33,7 +33,7 @@ public class StringProperty extends Property {
     public static class Maker extends PropertyMaker {
 
         /**
-         * @param propName name of property for which to create a Maker
+         * @param propId the id of the property for which a Maker should be created
          */
         public Maker(int propId) {
             super(propId);
index a8cb201d377851789ad85fb2e264e01058f06fa9..554501c73c2f00f946c8e4df89549b338b8da1f9 100644 (file)
@@ -78,10 +78,7 @@ public class TableColLength extends LengthProperty {
     }
 
     /**
-     * Return the value of this Numeric.
-     * @param context Evaluation context
-     * @return the value
-     * @see Numeric#getNumericValue(Object)
+     * @see org.apache.fop.datatypes.Numeric#getNumericValue(PercentBaseContext)
      */
     public double getNumericValue(PercentBaseContext context) {
         return getNumericValue();
@@ -96,9 +93,7 @@ public class TableColLength extends LengthProperty {
     }
 
     /**
-     * Returns the length in 1/1000ths of a point (millipoints)
-     * @param Evaluation context
-     * @return the length in millipoints
+     * @see org.apache.fop.datatypes.Numeric#getValue(PercentBaseContext)
      */
     public int getValue(PercentBaseContext context) {
         return getValue();
index 93942f63f2dc4a963e1ff2e92f4a241a20c5fef6..29f9aaea94a5958c7ac3e355e7a88e48ad53cb30 100644 (file)
@@ -164,10 +164,9 @@ public class FontSetup {
     }
 
     /**
-     * Add fonts from configuration file starting with
-     * internalnames F<num>
+     * Add fonts from configuration file starting with internal name F<num>.
      * @param fontInfo the font info object to set up
-     * @param fontInfos ???
+     * @param fontInfoList
      * @param num starting index for internal font numbering
      */
     public static void addConfiguredFonts(FontInfo fontInfo, List fontInfoList, int num) {
index f6a327171d868f76bf63dbf8619748cfa997cc2d..b01e9d075af894b40aadeb7340612ffcc32e01bf 100644 (file)
@@ -82,7 +82,7 @@ public abstract class BlockStackingLayoutManager extends AbstractLayoutManager
     private int contentAreaIPD = 0;
     
     /**
-     * @param the fo this LM deals with
+     * @param node the fo this LM deals with
      */
     public BlockStackingLayoutManager(FObj node) {
         super(node);
index 74db9963440efab41da165cf24a6f3c3aefa7152..cef81aaab021bea04e5f0ce25568c1c81b40cf56 100644 (file)
@@ -38,7 +38,6 @@ public abstract class KnuthElement {
      * Create a new KnuthElement.
      * This class being abstract, this can be called only by subclasses.
      *
-     * @param t    the type of this element (one of the KNUTH_* constants)
      * @param w    the width of this element
      * @param pos  the Position stored in this element
      * @param bAux is this an auxiliary element?
index ec36508f21421204edcb75fe3f1c538a8e54a948..7bf854715e9bfb0f26d5734d19efcc7ea46a7d16 100644 (file)
@@ -44,7 +44,6 @@ public abstract class AbstractGraphicsLayoutManager extends LeafNodeLayoutManage
     /**
      * Constructor
      * @param node the formatting object that creates this area
-     * @param parent the parent layout manager
      */
     public AbstractGraphicsLayoutManager(AbstractGraphics node) {
         super(node);
@@ -242,7 +241,7 @@ public abstract class AbstractGraphicsLayoutManager extends LeafNodeLayoutManage
      * to take the border/padding into account as well.
      * @param area the inline area to be updated
      * @param context the layout context used for adding the area
-     * @see LeafNodeLayoutManager#offsetArea(InlineArae, LayoutContext)
+     * @see LeafNodeLayoutManager#offsetArea(InlineArea, LayoutContext)
      */
     protected void offsetArea(InlineArea area, LayoutContext context) {
         int bpd = area.getBPD()
index a8decc5de41b008207ff1641777c2557ae363682..df074647599b8dcee22e293b9ba4e99726ec1118 100644 (file)
@@ -74,7 +74,7 @@ public class ContentLayoutManager extends AbstractBaseLayoutManager
         this.parentLM = pslm;
         holder = new LineArea();
 
-        setUserAgent(foTitle.getUserAgent());
+        //        setUserAgent(foTitle.getUserAgent());
 
         // use special layout manager to add the inline areas
         // to the Title.
@@ -186,18 +186,20 @@ public class ContentLayoutManager extends AbstractBaseLayoutManager
      *
      * @param ua the user agent
      */
-    public void setUserAgent(FOUserAgent ua) {
-        userAgent = ua;
-    }
+//     public void setUserAgent(FOUserAgent ua) {
+//         userAgent = ua;
+//     }
 
     /**
      * @see org.apache.fop.layoutmgr.LayoutManager#getUserAgent()
      */
-    public FOUserAgent getUserAgent() {
-        return userAgent;
-    }
+//     public FOUserAgent getUserAgent() {
+//         return userAgent;
+//     }
 
-    /** @see org.apache.fop.layoutmgr.LayoutManager */
+    /**
+     * @see org.apache.fop.layoutmgr.LayoutManager#setParent(LayoutManager)
+     */
     public void setParent(LayoutManager lm) {
         parentLM = lm;
     }
@@ -206,30 +208,36 @@ public class ContentLayoutManager extends AbstractBaseLayoutManager
         return this.parentLM;
     }
 
-    /** @see org.apache.fop.layoutmgr.LayoutManager */
+    /**
+     * @see org.apache.fop.layoutmgr.LayoutManager#isFinished()
+     */
     public boolean isFinished() {
         return false;
     }
 
-    /** @see org.apache.fop.layoutmgr.LayoutManager */
+    /**
+     * @see org.apache.fop.layoutmgr.LayoutManager#setFinished(boolean)
+     */
     public void setFinished(boolean isFinished) {
         //to be done
     }
 
-    /** @see org.apache.fop.layoutmgr.LayoutManager */
+    /**
+     * @see org.apache.fop.layoutmgr.LayoutManager#resetPosition(Position)
+     */
     public void resetPosition(Position position) {
         //to be done
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.LayoutManager#createNextChildLMs
+     * @see org.apache.fop.layoutmgr.LayoutManager#createNextChildLMs(int)
      */
     public boolean createNextChildLMs(int pos) {
         return false;
     }
 
     /**
-     * @see org.apache.fop.layoutmgr.LayoutManager#getChildLMs
+     * @see org.apache.fop.layoutmgr.LayoutManager#getChildLMs()
      */
     public List getChildLMs() {
         List childLMs = new ArrayList(1);
index 3463e38831133c47b385925cf0efb56e62f35d39..72bab19ffb0e5c6596728cd3aef3489a6d9014b3 100755 (executable)
@@ -117,7 +117,7 @@ public class InlineLayoutManager extends InlineStackingLayoutManager
         return inlineProps.spaceEnd;
     }
     
-    /** @see org.apache.fop.layoutmgr.inline.InlineStackingLayoutManager#createArea() */
+    /** @see org.apache.fop.layoutmgr.inline.InlineLayoutManager#createArea(boolean) */
     protected InlineArea createArea(boolean bInlineParent) {
         InlineArea area;
         if (bInlineParent) {
index 0e3667798c9338f4171bc861249d8d7e81f4f6f1..50fc89f84f71909221a2209e686686f65f0b9718 100644 (file)
@@ -380,8 +380,9 @@ public class TextLayoutManager extends LeafNodeLayoutManager {
      * @param str the string for the TextArea
      * @param width the MinOptMax width of the content
      * @param adjust the total ipd adjustment with respect to the optimal width
-     * @param base the baseline position
-     * @return the new word area
+     * @param context the layout context
+     * @param spaceDiff unused
+     * @return the new text area
      */
     protected TextArea createTextArea(String str, MinOptMax width, int adjust,
                                       LayoutContext context, int spaceDiff) {
index f21275acf4fa0b6b774dad133a900ba62275d396..db157f4cfbfabe3cc3470914a39eef171c9b140a 100644 (file)
@@ -73,7 +73,6 @@ public class ListBlockLayoutManager extends BlockStackingLayoutManager {
     /**
      * Create a new list block layout manager.
      * @param node list-block to create the layout manager for
-     * @param parent the parent layout manager
      */
     public ListBlockLayoutManager(ListBlock node) {
         super(node);
index c64bfc1b4fd9fd47eef0ca671c5c148021708414..9cbbb3d8bc24f5e97e6299f08c06b7c733e4e66e 100644 (file)
@@ -48,12 +48,13 @@ public abstract class CollapsingBorderModel {
     //public static final int FIRST_ROW_IN_GROUP      = 4;
     /** Indicates that the cell is/end in the last row of a body/table-header/table-footer */
     //public static final int LAST_ROW_IN_GROUP       = 8;
-    
+
+    // TODO: this is not MT safe. Must be removed.
     private static CollapsingBorderModel collapse = null;
     private static CollapsingBorderModel collapseWithPrecedence = null;
     
     /**
-     * @param cellLM the cell
+     * @param borderCollapse border collapse control
      * @return the border model for the cell
      */
     public static CollapsingBorderModel getBorderModelFor(int borderCollapse) {
index 529ec8e41f156d9575383d9103ac4654ef9f648a..bb4112483e6f2d33a9ac64e0c595632d6b2442c0 100644 (file)
@@ -1135,7 +1135,7 @@ public class TableContentLayoutManager implements PercentBaseContext {
     // --------- Property Resolution related functions --------- //
     
     /**
-     * @see org.apache.fop.datatypes.PercentBaseContext#getBaseLength(int, fobj)
+     * @see org.apache.fop.datatypes.PercentBaseContext#getBaseLength(int, FObj)
      */
     public int getBaseLength(int lengthBase, FObj fobj) {
         return tableLM.getBaseLength(lengthBase, fobj);
index 42c52c6ea8e25ff5893269a635d94264c577f4a5..a2257c310de9754113f2cec974b74199a87d131f 100644 (file)
@@ -76,7 +76,6 @@ public class PDFEncryptionManager {
      * the necessary cryptographic support is available.
      * @param params the PDF encryption params or null to disable encryption
      * @param pdf the PDF document to setup encryption for
-     * @param log the logger to send warnings to
      */
     public static void setupPDFEncryption(PDFEncryptionParams params, 
                                           PDFDocument pdf) {
index 0606243846dec1df88f2ae3f8ef2df4c0dc842e2..0230f9e453ba708544013c2cb73883c78391714a 100644 (file)
@@ -204,22 +204,31 @@ public abstract class Java2DRenderer extends AbstractRenderer implements Printab
         }
     }
 
-    /** @return The 0-based current page number */
+    /**
+     * @return The 0-based current page number
+     */
     public int getCurrentPageNumber() {
         return currentPageNumber;
     }
 
-    /** @param The 0-based current page number */
+    /**
+     * @param c the 0-based current page number
+     */
     public void setCurrentPageNumber(int c) {
         this.currentPageNumber = c;
     }
 
-    /** @return The 0-based total number of rendered pages  */
+    /**
+     * @return The 0-based total number of rendered pages
+     */
     public int getNumberOfPages() {
             return numberOfPages;
     }
 
-    /** clears the ViewportList, in case the document is reloaded */
+    /**
+     * Clears the ViewportList.
+     * Used if the document is reloaded.
+     */
     public void clearViewportList() {
         pageViewportList.clear();
         setCurrentPageNumber(0);
@@ -238,7 +247,8 @@ public abstract class Java2DRenderer extends AbstractRenderer implements Printab
      */
     public void renderPage(PageViewport pageViewport) 
                 throws IOException, FOPException {
-        pageViewportList.add(pageViewport.clone()); // FIXME clone
+        // TODO clone
+        pageViewportList.add(pageViewport.clone());
         currentPageNumber++;
     }
 
@@ -547,7 +557,7 @@ public abstract class Java2DRenderer extends AbstractRenderer implements Printab
      * Draw the background and borders. This draws the background and border
      * traits for an area given the position.
      *
-     * @param block the area to get the traits from
+     * @param area the area whose traits are used
      * @param startx the start x position
      * @param starty the start y position
      * @param width the width of the area
index ecaf4cc64ea834af44004ceeeda37ee37b469c84..6905a7cc15d711e222a527ae74423b1c3c1d672a 100644 (file)
@@ -46,9 +46,8 @@ public class SVGUserAgent extends UserAgentAdapter {
 
     /**
      * Creates a new SVGUserAgent.
-     * @param log an Commons logging instance
-     * @param pixelUnitToMM The pixel to millimeter conversion factor 
-     *                  currently in effect
+     * @param pixelUnitToMM the pixel to millimeter conversion factor
+     * currently in effect
      * @param at the current transform
      */
     public SVGUserAgent(float pixelUnitToMM, AffineTransform at) {
index 175cb6f0a01bfe87d2cd1df252f96b67f6531022..53cdf555bb977ce62a239b564520e759fc39f0d7 100644 (file)
@@ -80,14 +80,16 @@ public class FileCompare {
 
     /**
      * Compares two files to see if they are equal
-     * @param true if files are same, false otherwise
+     * @param f1 first file to compare
+     * @param f2 second file to compare
+     * @return true if files are same, false otherwise
      */
     public static boolean compareFiles(File f1, File f2) throws IOException {
         return (compareFileSize(f1, f2) && compareBytes(f1, f2));
     }
 
     /**
-     * Does a byte compare of two files
+     * Compare the contents of two files.
      * @param true if files are same byte-by-byte, false otherwise
      */
     private static boolean compareBytes(File file1, File file2) throws IOException {
index 7f61a1297e9cf75e34d474de0a2a432cf6b652bb..ad3259ea762b60d3b92749b6cc8469bbcc4b1a9e 100644 (file)
@@ -55,17 +55,16 @@ public class MinOptMax implements java.io.Serializable, Cloneable {
      * @param max the maximum value
      */
     public MinOptMax(int min, int opt, int max) {
+        // TODO: assert min<=opt<=max
         this.min = min;
         this.opt = opt;
         this.max = max;
     }
 
     /**
-     * New min/opt/max with the three values.
+     * Copy constructor.
      *
-     * @param min the minimum value
-     * @param opt the optimum value
-     * @param max the maximum value
+     * @param op the MinOptMax object to copy
      */
     public MinOptMax(MinOptMax op) {
         this.min = op.min;
@@ -73,6 +72,7 @@ public class MinOptMax implements java.io.Serializable, Cloneable {
         this.max = op.max;
     }
 
+    // TODO: remove this.
     /**
      * @see java.lang.Object#clone()
      */
@@ -114,6 +114,7 @@ public class MinOptMax implements java.io.Serializable, Cloneable {
      * @return MinOptMax new instance
      */
     public static MinOptMax multiply(MinOptMax op1, double mult) {
+        // TODO: assert mult>0
         return new MinOptMax((int)(op1.min * mult),
                              (int)(op1.opt * mult), (int)(op1.max * mult));
     }
@@ -129,18 +130,21 @@ public class MinOptMax implements java.io.Serializable, Cloneable {
     }
 
     /**
-     * Adds another MinOptMax instance to this one.
-     * @param op the other instance
+     * Adds min, opt and max to their counterpart components.
+     * @param min the value to add to the minimum value
+     * @param opt the value to add to the optimum value
+     * @param max the value to add to the maximum value
      */
     public void add(int min, int opt, int max) {
         this.min += min;
         this.opt += opt;
         this.max += max;
+        // TODO: assert min<=opt<=max
     }
 
     /**
-     * Adds another MinOptMax instance to this one.
-     * @param op the other instance
+     * Adds a length to all components.
+     * @param len the length to add
      */
     public void add(int len) {
         this.min += len;
@@ -150,7 +154,7 @@ public class MinOptMax implements java.io.Serializable, Cloneable {
 
 
     /**
-     * Subtracts from this instance using another.
+     * Subtracts another MinOptMax instance from this one.
      * @param op the other instance
      */
     public void subtract(MinOptMax op) {