]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Fixed various javadoc problems, fixed a deprecation warning, removed functions not...
authorManuel Mall <manuel@apache.org>
Wed, 26 Oct 2005 16:13:01 +0000 (16:13 +0000)
committerManuel Mall <manuel@apache.org>
Wed, 26 Oct 2005 16:13:01 +0000 (16:13 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@328666 13f79535-47bb-0310-9956-ffa450edef68

18 files changed:
src/java/org/apache/fop/datatypes/LengthBase.java
src/java/org/apache/fop/fo/flow/AbstractGraphics.java
src/java/org/apache/fop/fo/flow/ExternalGraphic.java
src/java/org/apache/fop/fo/flow/InstreamForeignObject.java
src/java/org/apache/fop/fo/properties/BoxPropShorthandParser.java
src/java/org/apache/fop/fo/properties/ToBeImplementedProperty.java
src/java/org/apache/fop/layoutmgr/LayoutContext.java
src/java/org/apache/fop/layoutmgr/inline/AbstractGraphicsLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/BasicScaledBaselineTable.java
src/java/org/apache/fop/layoutmgr/inline/CharacterLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/LeaderLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/LeafNodeLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/PageNumberCitationLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/PageNumberLayoutManager.java
src/java/org/apache/fop/layoutmgr/inline/TextLayoutManager.java
src/java/org/apache/fop/render/awt/viewer/ImageProxyPanel.java
src/java/org/apache/fop/render/ps/PSRenderer.java
src/java/org/apache/fop/tools/TestConverter.java

index 00f03c4507a9cf02f7f4897ecfadf3f5b9ceb143..30a93e51592f282b3222180a015529825ad9240c 100644 (file)
@@ -67,7 +67,7 @@ public class LengthBase implements PercentBase {
     /**
      * One of the defined types of LengthBase
      */
-    private /* final */ int iBaseType;
+    private /* final */ int baseType;
 
     /** For percentages based on other length properties */
     private Length baseLength;
@@ -76,13 +76,13 @@ public class LengthBase implements PercentBase {
      * Constructor
      * @param parentFO parent FO for this
      * @param plist property list for this
-     * @param iBaseType a member of {@link #PERCENT_BASED_LENGTH_TYPES}
+     * @param baseType a constant defining the type of teh percent base
      * @throws PropertyException In case an problem occurs while evaluating values
      */
-    public LengthBase(FObj parentFO, PropertyList plist, int iBaseType) throws PropertyException {
+    public LengthBase(FObj parentFO, PropertyList plist, int baseType) throws PropertyException {
         this.fobj = plist.getFObj();
-        this.iBaseType = iBaseType;
-        switch (iBaseType) {
+        this.baseType = baseType;
+        switch (baseType) {
         case FONTSIZE:
             this.baseLength = plist.get(Constants.PR_FONT_SIZE).getLength();
             break;
@@ -116,10 +116,10 @@ public class LengthBase implements PercentBase {
     public int getBaseLength(PercentBaseContext context) throws PropertyException {
         int baseLen = 0;
         if (context != null) {
-            if (iBaseType == FONTSIZE || iBaseType == INH_FONTSIZE) {
+            if (baseType == FONTSIZE || baseType == INH_FONTSIZE) {
                 return baseLength.getValue(context);
             }
-            baseLen =  context.getBaseLength(iBaseType,  fobj);
+            baseLen =  context.getBaseLength(baseType,  fobj);
         } else {
             fobj.getLogger().error("getBaseLength called without context");
         }
index 493738b12c73c638b8b2a4f075c09526ad6d0c90..540f2e616311335cbe1a21357664a46a6c75daa8 100644 (file)
@@ -265,12 +265,12 @@ public abstract class AbstractGraphics extends FObj {
     }
     
     /**
-     * @see org.apache.fop.fo.IntrinsicSizeAccess#getIntrinsicWidth()
+     * @return the graphics intrinsic width
      */
     public abstract int getIntrinsicWidth();
 
     /**
-     * @see org.apache.fop.fo.IntrinsicSizeAccess#getIntrinsicHeight()
+     * @return the graphics intrinsic height
      */
     public abstract int getIntrinsicHeight();
 }
index 5fa54cef480ea44e5f05af099b9b0616b080ecdf..f761918ca008b3750de4302a23d32efe9df274d8 100644 (file)
@@ -119,7 +119,7 @@ public class ExternalGraphic extends AbstractGraphics {
     }
 
     /**
-     * @see org.apache.fop.fo.IntrinsicSizeAccess#getIntrinsicWidth()
+     * @see org.apache.fop.fo.flow.AbstractGraphics#getIntrinsicWidth()
      */
     public int getIntrinsicWidth() {
         if (fopimage != null) {
@@ -130,7 +130,7 @@ public class ExternalGraphic extends AbstractGraphics {
     }
 
     /**
-     * @see org.apache.fop.fo.IntrinsicSizeAccess#getIntrinsicHeight()
+     * @see org.apache.fop.fo.flow.AbstractGraphics#getIntrinsicHeight()
      */
     public int getIntrinsicHeight() {
         if (fopimage != null) {
index e59015947a1ae6a710ff2f712260fa72e3bbed66..5b0f76d221b6b0150777fa73592b26a1425215cb 100644 (file)
@@ -109,7 +109,7 @@ public class InstreamForeignObject extends AbstractGraphics {
     }
 
     /**
-     * @see org.apache.fop.fo.IntrinsicSizeAccess#getIntrinsicWidth()
+     * @see org.apache.fop.fo.flow.AbstractGraphics#getIntrinsicWidth()
      */
     public int getIntrinsicWidth() {
         prepareIntrinsicSize();
@@ -121,7 +121,7 @@ public class InstreamForeignObject extends AbstractGraphics {
     }
 
     /**
-     * @see org.apache.fop.fo.IntrinsicSizeAccess#getIntrinsicHeight()
+     * @see org.apache.fop.fo.flow.AbstractGraphics#getIntrinsicHeight()
      */
     public int getIntrinsicHeight() {
         prepareIntrinsicSize();
index 9d99c0d1173c5879f1b8b50303f9f26343e10ff2..9d796363590b708f7cf99738f300565e6915227f 100644 (file)
@@ -37,7 +37,7 @@ public class BoxPropShorthandParser extends GenericShorthandParser {
      * Stores 1 to 4 values of same type.
      * 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(
+     * @see org.apache.fop.fo.properties.GenericShorthandParser#convertValueForProperty(
      * int, Property, PropertyMaker, PropertyList)
      */
     protected Property convertValueForProperty(int propId,
index c5466e2ac9a90a6c1690d21997fc35d88275f915..5a6fb481088a6cd7a78e64ef5957f3dab9f7b385 100644 (file)
@@ -43,7 +43,7 @@ public class ToBeImplementedProperty extends Property {
 
     /**
      * Constructor
-     * @param propName name of Property
+     * @param propId id of Property
      */
     public ToBeImplementedProperty(int propId) {
 
index e4710b1a16b6e1465ddc208d4789f86144441e3b..ef5b818c36ddb7fe5ff6ec6e7f12211d76b4f0a8 100644 (file)
@@ -254,7 +254,7 @@ public class LayoutContext {
     
     /**
      * @return the pending border and padding elements at the after edge
-     * @see addPendingAfterMark(BorderOrPaddingElement)
+     * @see #addPendingAfterMark(UnresolvedListElementWithLength)
      */
     public List getPendingAfterMarks() {
         if (this.pendingAfterMarks != null) {
@@ -279,7 +279,7 @@ public class LayoutContext {
     
     /**
      * @return the pending border and padding elements at the before edge
-     * @see addPendingBeforeMark(BorderOrPaddingElement)
+     * @see #addPendingBeforeMark(UnresolvedListElementWithLength)
      */
     public List getPendingBeforeMarks() {
         if (this.pendingBeforeMarks != null) {
index 8ebe812fd8e92940ed944a89244baf6c77b69e32..0f7b35aac5208385ee36009b1146ece32e014cf1 100644 (file)
@@ -208,7 +208,10 @@ public abstract class AbstractGraphicsLayoutManager extends LeafNodeLayoutManage
         vp.setOffset(0);
 
         // Common Border, Padding, and Background Properties
-        TraitSetter.addBorders(vp, fobj.getCommonBorderPaddingBackground(), this);
+        TraitSetter.addBorders(vp, fobj.getCommonBorderPaddingBackground()
+                                , false, false, false, false, this);
+        TraitSetter.addPadding(vp, fobj.getCommonBorderPaddingBackground()
+                                , false, false, false, false, this);
         TraitSetter.addBackground(vp, fobj.getCommonBorderPaddingBackground(), this);
 
         return vp;
@@ -225,7 +228,7 @@ public abstract class AbstractGraphicsLayoutManager extends LeafNodeLayoutManage
     }
     
     /**
-     * @see LeafNodeLayoutManager.makeAlignmentContext(LayoutContext)
+     * @see LeafNodeLayoutManager#makeAlignmentContext(LayoutContext)
      */
     protected AlignmentContext makeAlignmentContext(LayoutContext context) {
         return new AlignmentContext(
index db2b35e22e110a54b99f08ff93683d0f08900a17..31d659059b2d6aab1c8155f63a7b5086af547c6f 100644 (file)
@@ -180,7 +180,7 @@ public class BasicScaledBaselineTable implements ScaledBaselineTable, Constants
     }
 
     /**
-     * @see ScaledBaselineTable#getScaledBaselineTable(int)
+     * @see ScaledBaselineTable#deriveScaledBaselineTable(int)
      */
     public ScaledBaselineTable deriveScaledBaselineTable(int baselineIdentifier) {
         BasicScaledBaselineTable bac 
index 4d3a9adc25daa01745e3b8c0ea8ea76c39ef79b7..c337548fd61330e2f241b5edfdb16bb61fffcd4e 100644 (file)
@@ -52,14 +52,14 @@ public class CharacterLayoutManager extends LeafNodeLayoutManager {
      * Constructor
      *
      * @param node the fo:character formatting object
-     * @todo better null checking of node
      */
     public CharacterLayoutManager(Character node) {
+        // @todo better null checking of node
         super(node);
         fobj = node;
     }
     
-    /** @see LayoutManager#initialize */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#initialize */
     public void initialize() {
         font = fobj.getCommonFont().getFontState(fobj.getFOEventHandler().getFontInfo(), this);
         SpaceVal ls = SpaceVal.makeLetterSpacing(fobj.getLetterSpacing());
@@ -80,7 +80,7 @@ public class CharacterLayoutManager extends LeafNodeLayoutManager {
         return ch;
     }
 
-    /** @see LayoutManager#getNextKnuthElements(LayoutContext, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getNextKnuthElements(LayoutContext, int) */
     public LinkedList getNextKnuthElements(LayoutContext context, int alignment) {
         MinOptMax ipd;
         curArea = get(context);
@@ -162,7 +162,7 @@ public class CharacterLayoutManager extends LeafNodeLayoutManager {
         hc.updateOffset(1);
     }
 
-    /** @see InlineLevelLayoutManager#applyChanges(list) */
+    /** @see InlineLevelLayoutManager#applyChanges(List) */
     public boolean applyChanges(List oldList) {
         setFinished(false);
         if (isSomethingChanged) {
@@ -175,7 +175,7 @@ public class CharacterLayoutManager extends LeafNodeLayoutManager {
         }
     }
 
-    /** @see LayoutManager#getChangedKnuthElements(List, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getChangedKnuthElements(List, int) */
     public LinkedList getChangedKnuthElements(List oldList, int alignment) {
         if (isFinished()) {
             return null;
index 502d3ed6f7acd6c23ca5083cd4881af20a3c3941..d7983a9f602af16808df8a11c337393b33c77fc2 100644 (file)
@@ -63,7 +63,7 @@ public class LeaderLayoutManager extends LeafNodeLayoutManager {
         fobj = node;
     }
     
-    /** @see LayoutManager#initialize */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#initialize */
     public void initialize() {
         font = fobj.getCommonFont().getFontState(fobj.getFOEventHandler().getFontInfo(), this);
         // the property leader-alignment does not affect vertical positioning
@@ -213,7 +213,7 @@ public class LeaderLayoutManager extends LeafNodeLayoutManager {
         }
     }
 
-    /** @see LayoutManager#getNextKnuthElements(LayoutContext, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getNextKnuthElements(LayoutContext, int) */
     public LinkedList getNextKnuthElements(LayoutContext context,
                                            int alignment) {
         MinOptMax ipd;
@@ -266,13 +266,13 @@ public class LeaderLayoutManager extends LeafNodeLayoutManager {
         super.hyphenate(pos, hc);
     }
 
-    /** @see InlineLevelLayoutManager#applyChanges(list) */
+    /** @see InlineLevelLayoutManager#applyChanges(List) */
     public boolean applyChanges(List oldList) {
         setFinished(false);
         return false;
     }
 
-    /** @see LayoutManager#getNextKnuthElements(LayoutContext, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getChangedKnuthElements(List, int) */
     public LinkedList getChangedKnuthElements(List oldList,
                                               int alignment) {
         if (isFinished()) {
index ef5bee9fa008fcf44b3fd523905d9d28a27bed39..f4062a79a52d3114929bbbee961b400dd1be619f 100644 (file)
@@ -247,7 +247,7 @@ public abstract class LeafNodeLayoutManager extends AbstractLayoutManager
         area.setIPD(width);
     }
     
-    /** @see LayoutManager#getNextKnuthElements(LayoutContext, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getNextKnuthElements(LayoutContext, int) */
     public LinkedList getNextKnuthElements(LayoutContext context, int alignment) {
         curArea = get(context);
         
@@ -311,7 +311,7 @@ public abstract class LeafNodeLayoutManager extends AbstractLayoutManager
         return false;
     }
 
-    /** @see LayoutManager#getChangedKnuthElements(List, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getChangedKnuthElements(List, int) */
     public LinkedList getChangedKnuthElements(List oldList,
                                               int alignment) {
         if (isFinished()) {
index 9badf69863490534ec47ea3d1d7f88aaf3bc380d..e1c8c29d6060a342766a5f3b40486df6715658ad 100644 (file)
@@ -61,7 +61,7 @@ public class PageNumberCitationLayoutManager extends LeafNodeLayoutManager {
     }
 
     /**
-     * @see LeafNodeLayoutManager.makeAlignmentContext(LayoutContext)
+     * @see LeafNodeLayoutManager#makeAlignmentContext(LayoutContext)
      */
     protected AlignmentContext makeAlignmentContext(LayoutContext context) {
         return new AlignmentContext(
@@ -92,11 +92,6 @@ public class PageNumberCitationLayoutManager extends LeafNodeLayoutManager {
         }
     }
     
-    /** @see org.apache.fop.layoutmgr.inline.LeafNodeLayoutManager#getLead() */
-    public int getLead() {
-        return font.getAscender();
-    }
-
     /**
      * if id can be resolved then simply return a word, otherwise
      * return a resolvable area
@@ -145,7 +140,7 @@ public class PageNumberCitationLayoutManager extends LeafNodeLayoutManager {
         return width;
     }
 
-    /** @see org.apache.fop.layoutmgr.inline.LeafLayoutManager#addId */
+    /** @see org.apache.fop.layoutmgr.inline.LeafNodeLayoutManager#addId() */
     protected void addId() {
         getPSLM().addIDToPage(fobj.getId());
     }
index 2cabaa8a3432ce960de984a369fe7ee6b57f2b2a..7cbcd77c48b9166b5626e6140db5595ef0eb0ca3 100644 (file)
@@ -89,11 +89,6 @@ public class PageNumberLayoutManager extends LeafNodeLayoutManager {
         return text;
     }
     
-    /** @see org.apache.fop.layoutmgr.inline.LeafNodeLayoutManager#getLead() */
-    public int getLead() {
-        return font.getAscender();
-    }
-    
     /** @see org.apache.fop.layoutmgr.inline.LeafNodeLayoutManager#getEffectiveArea() */
     protected InlineArea getEffectiveArea() {
         TextArea baseArea = (TextArea)curArea;
@@ -133,7 +128,7 @@ public class PageNumberLayoutManager extends LeafNodeLayoutManager {
         return width;
     }
     
-    /** @see org.apache.fop.layoutmgr.inline.LeafLayoutManager#addId */
+    /** @see org.apache.fop.layoutmgr.inline.LeafNodeLayoutManager#addId() */
     protected void addId() {
         getPSLM().addIDToPage(fobj.getId());
     }
index 1755dd4fd3eb9b67811fc06b0ff2698dcf979584..f2e1cde33c0528bd9e17a8a8ef2cbe3f638fa2bd 100644 (file)
@@ -393,8 +393,6 @@ public class TextLayoutManager extends LeafNodeLayoutManager {
      * @param adjust the total ipd adjustment with respect to the optimal width
      * @param context the layout context
      * @param spaceDiff unused
-     * @param firstIndex the index of the first AreaInfo used for the TextArea
-     * @param lastIndex the index of the last AreaInfo used for the TextArea
      * @return the new text area
      */
     protected TextArea createTextArea(String str, MinOptMax width, int adjust,
@@ -429,7 +427,7 @@ public class TextLayoutManager extends LeafNodeLayoutManager {
         return textArea;
     }
 
-    /** @see LayoutManager#getNextKnuthElements(LayoutContext, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getNextKnuthElements(LayoutContext, int) */
     public LinkedList getNextKnuthElements(LayoutContext context, int alignment) {
         lineStartBAP = context.getLineStartBorderAndPaddingWidth();
         lineEndBAP = context.getLineEndBorderAndPaddingWidth();
@@ -688,7 +686,7 @@ public class TextLayoutManager extends LeafNodeLayoutManager {
         return bChanged;
     }
 
-    /** @see org.aapache.fop.layoutmgr.LayoutManager#getChangedKnuthElements(List, int) */
+    /** @see org.apache.fop.layoutmgr.LayoutManager#getChangedKnuthElements(List, int) */
     public LinkedList getChangedKnuthElements(List oldList,
                                               int alignment) {
         if (isFinished()) {
index 89cb27feb5d7acdd4500856427dc01b290ed2789..e9933b137fbe1db9d1b32f07087f7c60ae1df47e 100644 (file)
@@ -105,7 +105,9 @@ public class ImageProxyPanel extends JPanel {
     /**
      * Gets the image data and paints it on screen. Will make
      * calls to getPageImage as required.
-     * @see org.apache.fop.render.java2d.Java2DRenderer#getPageImage()
+     * @param graphics
+     * @see javax.swing.JComponent#paintComponent(Graphics)
+     * @see org.apache.fop.render.java2d.Java2DRenderer#getPageImage(int)
      */
     public synchronized void paintComponent(Graphics graphics) {
         try {
index 34a5be8cb1a06f1d6b38d96c8a6cab23751448fb..98599eeb42a401dc8345363f09a7f8aaf2e0e24f 100644 (file)
@@ -387,7 +387,8 @@ public class PSRenderer extends AbstractPathOrientedRenderer {
         }
     }
 
-    /** @see org.apache.fop.render.AbstractPathOrientedRenderer */
+    /** @see org.apache.fop.render.AbstractPathOrientedRenderer#drawBackAndBorders(
+     * Area, float, float, float, float) */
     protected void drawBackAndBorders(Area area, float startx, float starty,
             float width, float height) {
         if (area.hasTrait(Trait.BACKGROUND)
index 04bcac27547331d76fdd650a3ef49f9f0f3528a9..8025799952a590ec17bf63eb9ed7d4630cbd8020 100644 (file)
@@ -141,7 +141,7 @@ public class TestConverter {
 
     /**
      * Controls whether to set logging to debug level
-     * @param If true, debug level, if false, error level
+     * @param debug If true, debug level, if false, error level
      */
     public void setDebug(boolean debug) {
         if (debug) {