]> source.dussan.org Git - xmlgraphics-fop.git/commitdiff
Some cleanup of the AFP Renderer code (tab char removal and a few other things)....
authorManuel Mall <manuel@apache.org>
Fri, 28 Apr 2006 11:58:33 +0000 (11:58 +0000)
committerManuel Mall <manuel@apache.org>
Fri, 28 Apr 2006 11:58:33 +0000 (11:58 +0000)
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@397875 13f79535-47bb-0310-9956-ffa450edef68

19 files changed:
src/sandbox/META-INF/services/org.apache.fop.fo.ElementMapping [new file with mode: 0644]
src/sandbox/org/apache/fop/render/afp/AFPRenderer.java
src/sandbox/org/apache/fop/render/afp/exceptions/NestedRuntimeException.java
src/sandbox/org/apache/fop/render/afp/exceptions/RendererRuntimeException.java
src/sandbox/org/apache/fop/render/afp/extensions/AFPAttribute.java
src/sandbox/org/apache/fop/render/afp/extensions/AFPElementMapping.java
src/sandbox/org/apache/fop/render/afp/extensions/AFPPageSetup.java
src/sandbox/org/apache/fop/render/afp/fonts/AFPFont.java
src/sandbox/org/apache/fop/render/afp/fonts/CharacterSetOrientation.java
src/sandbox/org/apache/fop/render/afp/fonts/FopCharacterSet.java
src/sandbox/org/apache/fop/render/afp/fonts/OutlineFont.java
src/sandbox/org/apache/fop/render/afp/modca/IMImageObject.java
src/sandbox/org/apache/fop/render/afp/modca/ImageObject.java
src/sandbox/org/apache/fop/render/afp/modca/ImageRasterPattern.java
src/sandbox/org/apache/fop/render/afp/modca/ImageSegment.java
src/sandbox/org/apache/fop/render/afp/modca/PageObject.java
src/sandbox/org/apache/fop/render/afp/modca/PresentationTextData.java
src/sandbox/org/apache/fop/render/afp/tools/DTDEntityResolver.java
src/sandbox/org/apache/fop/render/afp/tools/StructuredFieldReader.java

diff --git a/src/sandbox/META-INF/services/org.apache.fop.fo.ElementMapping b/src/sandbox/META-INF/services/org.apache.fop.fo.ElementMapping
new file mode 100644 (file)
index 0000000..d38bb66
--- /dev/null
@@ -0,0 +1 @@
+org.apache.fop.render.afp.extensions.AFPElementMapping
\ No newline at end of file
index e4505d0504c1dc43fa757dae6ea9a7e4af040e9a..9cee8d12106e34f661462480ca486acb5e47fac9 100644 (file)
@@ -1157,9 +1157,7 @@ public class AFPRenderer extends AbstractPathOrientedRenderer {
                         }
                         convertToGrayScaleImage(io, fopimage.getBitmaps());
                     }
-                }
-                else
-                {
+                } else {
                     if (!fopimage.load(FopImage.BITMAP)) {
                         return;
                     }
@@ -1276,7 +1274,7 @@ public class AFPRenderer extends AbstractPathOrientedRenderer {
 
         // Create an AFPFontAttributes object from the current font details
         AFPFontAttributes afpFontAttributes =
-            new AFPFontAttributes(name, tf,_currentFontSize);
+            new AFPFontAttributes(name, tf, _currentFontSize);
 
         if (!_currentPageFonts.containsKey(afpFontAttributes.getFontKey())) {
             // Font not found on current page, so add the new one
@@ -1547,7 +1545,8 @@ public class AFPRenderer extends AbstractPathOrientedRenderer {
             || rotation == 270) {
             _portraitRotation = rotation;
         } else {
-            throw new IllegalArgumentException("The portrait rotation must be one of the values 0, 90, 180, 270");
+            throw new IllegalArgumentException("The portrait rotation must be one"
+                + " of the values 0, 90, 180, 270");
 
         }
 
@@ -1568,8 +1567,8 @@ public class AFPRenderer extends AbstractPathOrientedRenderer {
             || rotation == 270) {
             _landscapeRotation = rotation;
         } else {
-            throw new IllegalArgumentException("The landscape rotation must be one of the values 0, 90, 180, 270");
-
+            throw new IllegalArgumentException("The landscape rotation must be one"
+                + " of the values 0, 90, 180, 270");
         }
 
     }
@@ -1692,21 +1691,21 @@ public class AFPRenderer extends AbstractPathOrientedRenderer {
         }
 
         ViewPortPos(Rectangle2D view, CTM ctm) {
-            ViewPortPos currentVP = (ViewPortPos)viewPortPositions.get(viewPortPositions.size()-1);
+            ViewPortPos currentVP = (ViewPortPos)viewPortPositions.get(viewPortPositions.size() - 1);
             int xOrigin;
             int yOrigin;
             int width;
             int height;
             switch (currentVP.rot) {
                 case 90:
-                    width = mpts2units(view.getHeight());;
+                    width = mpts2units(view.getHeight());
                     height = mpts2units(view.getWidth());
                     xOrigin = _pageWidth - width - mpts2units(view.getY()) - currentVP.y;
                     yOrigin = mpts2units(view.getX()) + currentVP.x;
                     break;
                 case 180:
                     width = mpts2units(view.getWidth());
-                    height = mpts2units(view.getHeight());;
+                    height = mpts2units(view.getHeight());
                     xOrigin = _pageWidth - width - mpts2units(view.getX()) - currentVP.x;
                     yOrigin = _pageHeight - height - mpts2units(view.getY()) - currentVP.y;
                     break;
index 92254e6e607b58588b504ad8eb932c1bc6ec676b..3e8c1d9475940af1c64676d5bfe00935efc65824 100644 (file)
@@ -15,6 +15,7 @@
  */
 
 /* $Id$ */
+
 package org.apache.fop.render.afp.exceptions;
 
 import java.io.PrintStream;
@@ -28,81 +29,81 @@ import java.io.PrintWriter;
  *
  */
 public abstract class NestedRuntimeException extends RuntimeException {
-
-       /** Root cause of this nested exception */
-       private Throwable _underlyingException;
-
-       /**
-        * Construct a <code>NestedRuntimeException</code> with the specified detail message.
-        * @param msg The detail message.
-        */
-       public NestedRuntimeException(String msg) {
-               super(msg);
-       }
-
-       /**
-        * Construct a <code>NestedRuntimeException</code> with the specified
-        * detail message and nested exception.
-        * @param msg The detail message.
-        * @param t The nested exception.
-        */
-       public NestedRuntimeException(String msg, Throwable t) {
-               super(msg);
-               _underlyingException = t;
-
-       }
-
-       /**
-        * Gets the original triggering exception
-        * @return The original exception as a throwable.
-        */
-       public Throwable getUnderlyingException() {
-
-               return _underlyingException;
-
-       }
-
-       /**
-        * Return the detail message, including the message from the nested
-        * exception if there is one.
-        * @return The detail message.
-        */
-       public String getMessage() {
-
-               if (_underlyingException == null) {
-                       return super.getMessage();
-               } else {
-                       return super.getMessage()
-                               + "; nested exception is "
-                               + _underlyingException.getClass().getName();
-               }
-
-       }
-
-       /**
-        * Print the composite message and the embedded stack trace to the specified stream.
-        * @param ps the print stream
-        */
-       public void printStackTrace(PrintStream ps) {
-               if (_underlyingException == null) {
-                       super.printStackTrace(ps);
-               } else {
-                       ps.println(this);
-                       _underlyingException.printStackTrace(ps);
-               }
-       }
-
-       /**
-        * Print the composite message and the embedded stack trace to the specified writer.
-        * @param pw the print writer
-        */
-       public void printStackTrace(PrintWriter pw) {
-               if (_underlyingException == null) {
-                       super.printStackTrace(pw);
-               } else {
-                       pw.println(this);
-                       _underlyingException.printStackTrace(pw);
-               }
-       }
-
+    
+    /** Root cause of this nested exception */
+    private Throwable _underlyingException;
+    
+    /**
+     * Construct a <code>NestedRuntimeException</code> with the specified detail message.
+     * @param msg The detail message.
+     */
+    public NestedRuntimeException(String msg) {
+        super(msg);
+    }
+    
+    /**
+     * Construct a <code>NestedRuntimeException</code> with the specified
+     * detail message and nested exception.
+     * @param msg The detail message.
+     * @param t The nested exception.
+     */
+    public NestedRuntimeException(String msg, Throwable t) {
+        super(msg);
+        _underlyingException = t;
+        
+    }
+    
+    /**
+     * Gets the original triggering exception
+     * @return The original exception as a throwable.
+     */
+    public Throwable getUnderlyingException() {
+        
+        return _underlyingException;
+        
+    }
+    
+    /**
+     * Return the detail message, including the message from the nested
+     * exception if there is one.
+     * @return The detail message.
+     */
+    public String getMessage() {
+        
+        if (_underlyingException == null) {
+            return super.getMessage();
+        } else {
+            return super.getMessage()
+            + "; nested exception is "
+                + _underlyingException.getClass().getName();
+        }
+        
+    }
+    
+    /**
+     * Print the composite message and the embedded stack trace to the specified stream.
+     * @param ps the print stream
+     */
+    public void printStackTrace(PrintStream ps) {
+        if (_underlyingException == null) {
+            super.printStackTrace(ps);
+        } else {
+            ps.println(this);
+            _underlyingException.printStackTrace(ps);
+        }
+    }
+    
+    /**
+     * Print the composite message and the embedded stack trace to the specified writer.
+     * @param pw the print writer
+     */
+    public void printStackTrace(PrintWriter pw) {
+        if (_underlyingException == null) {
+            super.printStackTrace(pw);
+        } else {
+            pw.println(this);
+            _underlyingException.printStackTrace(pw);
+        }
+    }
+    
 }
index a3be63ecb03072d0b7b352ca464be0b8c4c6faf8..d04247196e309e9dbc4f41050002013b7b9d2de2 100644 (file)
@@ -23,23 +23,23 @@ package org.apache.fop.render.afp.exceptions;
  * <p/>
  */
 public class RendererRuntimeException extends NestedRuntimeException {
-
-       /**
-        * Constructs a RendererRuntimeException with the specified message.
-        * @param msg the exception mesaage
-        */
-       public RendererRuntimeException(String msg) {
-               super(msg);
-       }
-
-       /**
-        * Constructs a RendererRuntimeException with the specified message
-        * wrapping the underlying exception.
-        * @param msg the exception mesaage
-        * @param t the underlying exception
-        */
-       public RendererRuntimeException(String msg, Throwable t) {
-               super(msg, t);
-       }
-
+    
+    /**
+     * Constructs a RendererRuntimeException with the specified message.
+     * @param msg the exception mesaage
+     */
+    public RendererRuntimeException(String msg) {
+        super(msg);
+    }
+    
+    /**
+     * Constructs a RendererRuntimeException with the specified message
+     * wrapping the underlying exception.
+     * @param msg the exception mesaage
+     * @param t the underlying exception
+     */
+    public RendererRuntimeException(String msg, Throwable t) {
+        super(msg, t);
+    }
+    
 }
index a1c63856b8d70346ca14e4829089383f3346e221..421408902ce41a6a6b7b7b9e0b50ae0871173f4d 100755 (executable)
@@ -48,6 +48,7 @@ public class AFPAttribute extends StringProperty.Maker {
 
     /**
      * Overide the make method to return the property object
+     * @param propertyList the property list from which to make the property
      * @return property The property object.
      */
     public Property make(PropertyList propertyList) {
index 350b75c1e6a016f8825346ea0bb75407fa5149e4..53421021e42dd7071c48f6ca52a729752dfd256e 100755 (executable)
@@ -47,7 +47,7 @@ public class AFPElementMapping extends ElementMapping {
     /**
      * The namespace used for AFP extensions
      */
-    public static final String NAMESPACE = "http://org.apache.fop/extensions/afp";
+    public static final String NAMESPACE = "http://xmlgraphics.apache.org/fop/extensions/afp";
 
     /**
      * The usual namespace prefix used for AFP extensions
index d95a3a458619a87289333a3125bd9979833f1f72..a3984d2be06d6d224c20c22d0278ed456881363a 100644 (file)
@@ -71,7 +71,7 @@ public class AFPPageSetup implements ExtensionAttachment, Serializable {
 
     /**
      * Sets the value
-     * @param value The value name to set.
+     * @param source The value name to set.
      */
     public void setValue(String source) {
         this.value = source;
index d17536a8d25434d68c18cb7844314f7877c2e38d..b7f27d65e5705f75e00d96bc630283d6ea882d08 100644 (file)
@@ -76,6 +76,7 @@ public abstract class AFPFont extends Typeface {
     /**
      * Returns the character set for a given size
      * @param size the font size
+     * @return the character set object
      */
     public abstract CharacterSet getCharacterSet(int size);
 
index 68eb2e4130b69119ae3aafc8b9ea1dd387013624..084edfb5ed0454b00ea42b5cfaad2e0c077ab0ff 100644 (file)
@@ -178,7 +178,7 @@ public class CharacterSetOrientation {
     /**
      * XHeight refers to the height of the lower case letters above
      * the baseline.
-     * @return heightX the typical height of characters
+     * @return heightX the typical height of characters
      */
     public int getXHeight() {
         return _xHeight;
@@ -224,7 +224,7 @@ public class CharacterSetOrientation {
      * Descender depth is the distance from the character baseline to
      * the bottom of a character box. A negative descender depth signifies
      * that all of the graphic character is above the character baseline.
-     * @param descender        the descender value in millipoints
+     * @param descender the descender value in millipoints
      */
     public void setDescender(int descender) {
         _descender = descender;
@@ -250,7 +250,7 @@ public class CharacterSetOrientation {
      * Set the width (in 1/1000ths of a point size) of the character
      * identified by the parameter passed.
      * @param character the character for which the width is being set
-     * @param width    the widths of the character
+     * @param width the widths of the character
      */
     public void setWidth(int character, int width) {
 
index 635cef025992fe6e39dd00f0f45946c10b02d710..ff8f9240eda4bf499a1200facb5e2770034c39f4 100644 (file)
@@ -15,6 +15,7 @@
  */
 
 /* $Id$ */
+
 package org.apache.fop.render.afp.fonts;
 
 import org.apache.fop.fonts.Typeface;
@@ -35,7 +36,7 @@ public class FopCharacterSet extends CharacterSet {
      * @param encoding the encoding of the font
      * @param name the character set name
      * @param size the font size
-     * @param Typeface the fop character set
+     * @param characterSet the fop character set
      */
     public FopCharacterSet(
         String codePage,
index 00757f4ba107e4a0e2bd6fa7cf4be2754f0f1985..eabd3f631bac40c899c5b58d69bd4e42824991f8 100644 (file)
@@ -15,8 +15,8 @@
  */
 
 /* $Id$ */
-package org.apache.fop.render.afp.fonts;
 
+package org.apache.fop.render.afp.fonts;
 
 /**
  * A font defined as a set of lines and curves as opposed to a bitmap font. An
@@ -154,7 +154,7 @@ public class OutlineFont extends AFPFont {
     public int[] getWidths(int size) {
 
         int[] widths =  _characterSet.getWidths();
-        for (int i = 0 ; i < widths.length; i++) {
+        for (int i = 0; i < widths.length; i++) {
             widths[i] = widths[i] / 1000 * size;
         }
         return widths;
index aa510da6b7c93b55be9628a46a432c44aaa11848..9a6e195d1ef8297296b20d8a02f032ca2ec8b716 100644 (file)
@@ -59,7 +59,7 @@ public class IMImageObject extends AbstractNamedAFPObject {
     /**
      * Constructor for the image object with the specified name,
      * the name must be a fixed length of eight characters.
-     * @param name     The name of the image.
+     * @param name The name of the image.
      */
     public IMImageObject(String name) {
 
index 8e4e4c386bc1228285eec38959bc80c95f7c9294..fc027b8af127fe0c9ec50e6472ee298142179308 100644 (file)
@@ -42,7 +42,7 @@ public class ImageObject extends AbstractNamedAFPObject {
     /**
      * Constructor for the image object with the specified name,
      * the name must be a fixed length of eight characters.
-     * @param name     The name of the image.
+     * @param name The name of the image.
      */
     public ImageObject(String name) {
 
index c49f392b5c9e2450a87fc8627f23f23d8568ce10..673f04cad85cddb32832c12d5b66f8c27a49798c 100644 (file)
@@ -749,7 +749,7 @@ public class ImageRasterPattern {
                 break;
         }
 
-        for(int i = 0; i < repeat; i++) {
+        for (int i = 0; i < repeat; i++) {
 
             System.arraycopy(greypattern, 0, rasterdata, i * 32, 32);
 
index 0e5a2f691141c8668d1339d6b4a7646710db7923..9a27b79f7efbe5665b03225e1c3af78f85724f36 100644 (file)
@@ -65,7 +65,7 @@ public class ImageSegment extends AbstractAFPObject {
     /**
      * Constructor for the image segment with the specified name,
      * the name must be a fixed length of eight characters.
-     * @param name     The name of the image.
+     * @param name The name of the image.
      */
     public ImageSegment(String name) {
 
index 9bf543eee2d326e8b0c1130e88b59bb88c7d4cc9..102d7380e05292352da47b00be7cbbb7d0f8f875 100644 (file)
@@ -70,7 +70,7 @@ public class PageObject extends AbstractPageObject {
 
     /**
      * Adds an overlay to the page resources
-     * @param the overlay to add
+     * @param overlay the overlay to add
      */
     public void addOverlay(Overlay overlay) {
         if (_resourceGroup == null) {
index 38495c85960ed9ba65296de32e075bcded70467a..99b6f452d82ba9dbbd4c5e4b17790caa6ed98e6f 100644 (file)
@@ -488,18 +488,18 @@ public class PresentationTextData extends AbstractAFPObject {
         ByteArrayOutputStream afpdata) {
 
         afpdata.write(new byte[] {
-              15                // Control sequence length
-            , (byte)0x81         // Control sequence function type
-            , 0x00              // Reserved; must be zero
-            , 0x01              // Color space - 0x01 = RGB
-            , 0x00              // Reserved; must be zero
-            , 0x00              // Reserved; must be zero
-            , 0x00              // Reserved; must be zero
-            , 0x00              // Reserved; must be zero
-            , 8                         // Number of bits in component 1
-            , 8                         // Number of bits in component 2
-            , 8                         // Number of bits in component 3
-            , 0                         // Number of bits in component 4
+              15                // Control sequence length
+            , (byte)0x81        // Control sequence function type
+            , 0x00              // Reserved; must be zero
+            , 0x01              // Color space - 0x01 = RGB
+            , 0x00              // Reserved; must be zero
+            , 0x00              // Reserved; must be zero
+            , 0x00              // Reserved; must be zero
+            , 0x00              // Reserved; must be zero
+            , 8                 // Number of bits in component 1
+            , 8                 // Number of bits in component 2
+            , 8                 // Number of bits in component 3
+            , 0                 // Number of bits in component 4
             , (byte)(col.getRed())   // Red intensity
             , (byte)(col.getGreen()) // Green intensity
             , (byte)(col.getBlue())  // Blue intensity
index 94f4c46bd297e62b1ff53cf424f952af347ff5b8..ffdab85e4327d2e23079111f7079c0690f0b1b5f 100644 (file)
@@ -15,6 +15,7 @@
  */
 
 /* $Id$ */
+
 package org.apache.fop.render.afp.tools;
 
 import java.io.IOException;
@@ -67,11 +68,11 @@ public class DTDEntityResolver implements EntityResolver {
     throws IOException {
 
         URL resource = null;
-        if( AFP_DTD_1_2_ID.equals(publicId) ) {
+        if ( AFP_DTD_1_2_ID.equals(publicId) ) {
             resource = getResource( AFP_DTD_1_2_RESOURCE );
-        } else if( AFP_DTD_1_1_ID.equals(publicId) ) {
+        } else if ( AFP_DTD_1_1_ID.equals(publicId) ) {
             resource = getResource( AFP_DTD_1_1_RESOURCE );
-        } else if( AFP_DTD_1_0_ID.equals(publicId) ) {
+        } else if ( AFP_DTD_1_0_ID.equals(publicId) ) {
             throw new FontRuntimeException(
                 "The AFP Installed Font Definition 1.0 DTD is not longer supported" );
         } else if( systemId != null && systemId.indexOf("afp-fonts.dtd") >= 0 ) {
@@ -95,14 +96,14 @@ public class DTDEntityResolver implements EntityResolver {
      * @return the URL of the required resource
      * @throws FontRuntimeException if the resource could not be found.
      */
-    private URL getResource( String resourcePath ) {
+    private URL getResource(String resourcePath) {
         ClassLoader cl = Thread.currentThread().getContextClassLoader();
         if (cl == null) {
             cl = ClassLoader.getSystemClassLoader();
         }
 
         URL resource = cl.getResource( resourcePath );
-        if( resource == null ) {
+        if (resource == null) {
             throw new FontRuntimeException( "Resource " + resourcePath +
                 " could not be found on the classpath" );
         }
index 7df320c272621a69a3b2844629540945bc1742eb..4c1a52f3ea86d5ea2fe9876af3de72a9bc18b127 100644 (file)
@@ -58,6 +58,7 @@ public class StructuredFieldReader {
      * Get the next structured field as identified by the identifer
      * parameter (this must be a valid MO:DCA structured field.
      * @param identifier the three byte identifier
+     * @throws IOException
      */
     public byte[] getNext(byte[] identifier) throws IOException {
 
@@ -109,7 +110,8 @@ public class StructuredFieldReader {
                 length[0] = bufferData[b];
                 length[1] = bufferData[a];
 
-                int reclength = ((length[0] & 0xFF) << 8) + (length[1] & 0xFF)  - identifier.length -2;
+                int reclength = ((length[0] & 0xFF) << 8) 
+                                + (length[1] & 0xFF) - identifier.length - 2;
 
                 byte[] retval = new byte[reclength];