mbenson     2005/02/03 15:38:01

  Modified:    src/main/org/apache/tools/ant/taskdefs/optional/image
                        Image.java
  Log:
  Javadoc & changed File constructors from
  new File(f1.getAbsolutePath() + File.separator + f2.getName()) to
  new File(f1, f2.getName())
  
  Revision  Changes    Path
  1.15      +50 -63    
ant/src/main/org/apache/tools/ant/taskdefs/optional/image/Image.java
  
  Index: Image.java
  ===================================================================
  RCS file: 
/home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/optional/image/Image.java,v
  retrieving revision 1.14
  retrieving revision 1.15
  diff -u -r1.14 -r1.15
  --- Image.java        28 Jan 2005 16:11:40 -0000      1.14
  +++ Image.java        3 Feb 2005 23:38:01 -0000       1.15
  @@ -1,5 +1,5 @@
   /*
  - * Copyright  2002,2004-2005 The Apache Software Foundation
  + * Copyright  2002, 2004-2005 The Apache Software Foundation
    *
    *  Licensed under the Apache License, Version 2.0 (the "License");
    *  you may not use this file except in compliance with the License.
  @@ -37,40 +37,43 @@
   import java.util.Vector;
   
   /**
  - * A MatchingTask which relies on <A
  - * HREF="http://java.sun.com/products/java-media/jai";>JAI (Java
  - * Advanced Imaging)</A> to perform image manipulation operations on
  + * A MatchingTask which relies on <a
  + * href="http://java.sun.com/products/java-media/jai";>JAI (Java
  + * Advanced Imaging)</a> to perform image manipulation operations on
    * existing images.  The operations are represented as ImageOperation
    * DataType objects.  The operations are arranged to conform to the
  - * Chaining Model of JAI.  Check out the <A
  - * 
HREF="http://java.sun.com/products/java-media/jai/forDevelopers/jai1_0_1guide-unc/";>
  - * JAI Programming Guide</A>
  + * Chaining Model of JAI.  Check out the <a
  + * 
href="http://java.sun.com/products/java-media/jai/forDevelopers/jai1_0_1guide-unc/";>
  + * JAI Programming Guide</a>.
    *
    * @see org.apache.tools.ant.types.optional.image.ImageOperation
    * @see org.apache.tools.ant.types.DataType
    */
   public class Image extends MatchingTask {
       protected Vector instructions = new Vector();
  -    protected String str_encoding = "JPEG";
       protected boolean overwrite = false;
  -    protected boolean garbage_collect = false;
  -    private boolean failonerror = true;
       protected Vector filesets = new Vector();
  -
  -
       protected File srcDir = null;
       protected File destDir = null;
   
  +    //cannot remove underscores due to protected visibility >:(
  +    protected String str_encoding = "JPEG";
  +    protected boolean garbage_collect = false;
  +
  +    private boolean failonerror = true;
  +
       /**
  -     * Adds a set of files to be deleted.
  +     * Add a set of files to be deleted.
  +     * @param set the FileSet to add.
        */
       public void addFileset(FileSet set) {
           filesets.addElement(set);
       }
   
       /**
  +     * Set whether to fail on error.
        * If false, note errors to the output but keep going.
  -     * @param failonerror true or false
  +     * @param failonerror true or false.
        */
       public void setFailOnError(boolean failonerror) {
           this.failonerror = failonerror;
  @@ -78,55 +81,58 @@
   
       /**
        * Set the source dir to find the image files.
  +     * @param srcDir the directory in which the image files reside.
        */
       public void setSrcdir(File srcDir) {
           this.srcDir = srcDir;
       }
   
       /**
  -     * Set the image encoding type.  <A
  -     * 
HREF="http://java.sun.com/products/java-media/jai/forDevelopers/jai1_0_1guide-unc/Encode.doc.html#56610";>
  -     * See this table in the JAI Programming Guide</A>.
  +     * Set the image encoding type.  <a
  +     * 
href="http://java.sun.com/products/java-media/jai/forDevelopers/jai1_0_1guide-unc/Encode.doc.html#56610";>
  +     * See this table in the JAI Programming Guide</a>.
  +     * @param encoding the String image encoding.
        */
       public void setEncoding(String encoding) {
           str_encoding = encoding;
       }
   
       /**
  -     *  Sets whether or not to overwrite a file if there is a naming 
conflict.
  +     * Set whether to overwrite a file if there is a naming conflict.
  +     * @param overwrite whether to overwrite.
        */
       public void setOverwrite(boolean overwrite) {
           this.overwrite = overwrite;
       }
   
       /**
  -     *  Enables Garbage Collection after each image processed.
  -     *  Defaults to false.
  +     * Set whether to invoke Garbage Collection after each image processed.
  +     * Defaults to false.
  +     * @param gc whether to invoke the garbage collector.
        */
       public void setGc(boolean gc) {
           garbage_collect = gc;
       }
   
  -
       /**
  -     * Sets the destination directory for manipulated images.
  -     * @param destDir The destination directory
  +     * Set the destination directory for manipulated images.
  +     * @param destDir The destination directory.
        */
       public void setDestDir(File destDir) {
           this.destDir = destDir;
       }
   
       /**
  -     * Adds an ImageOperation to chain.
  -     * @param instr The ImageOperation to append to the chain
  +     * Add an ImageOperation to chain.
  +     * @param instr The ImageOperation to append to the chain.
        */
       public void addImageOperation(ImageOperation instr) {
           instructions.add(instr);
       }
   
       /**
  -     * Adds a Rotate ImageOperation to the chain
  -     * @param instr The Rotate operation to add to the chain
  +     * Add a Rotate ImageOperation to the chain.
  +     * @param instr The Rotate operation to add to the chain.
        * @see org.apache.tools.ant.types.optional.image.Rotate
        */
       public void addRotate(Rotate instr) {
  @@ -134,8 +140,8 @@
       }
   
       /**
  -     * Adds a Scale ImageOperation to the chain
  -     * @param instr The Scale operation to add to the chain
  +     * Add a Scale ImageOperation to the chain.
  +     * @param instr The Scale operation to add to the chain.
        * @see org.apache.tools.ant.types.optional.image.Scale
        */
       public void addScale(Scale instr) {
  @@ -143,9 +149,9 @@
       }
   
       /**
  -     * Adds a Draw ImageOperation to the chain.  DrawOperation
  -     * DataType objects can be nested inside the Draw object
  -     * @param instr The Draw operation to add to the chain
  +     * Add a Draw ImageOperation to the chain.  DrawOperation
  +     * DataType objects can be nested inside the Draw object.
  +     * @param instr The Draw operation to add to the chain.
        * @see org.apache.tools.ant.types.optional.image.Draw
        * @see org.apache.tools.ant.types.optional.image.DrawOperation
        */
  @@ -153,11 +159,9 @@
           instructions.add(instr);
       }
   
  -
       /**
  -    * Adds an ImageOperation to chain.
  -    * Adds an ImageOperation to the chain.
  -    * @param instr The ImageOperation to append to the chain
  +    * Add an ImageOperation to chain.
  +    * @param instr The ImageOperation to append to the chain.
       * @since Ant 1.7
       */
       public void add(ImageOperation instr) {
  @@ -167,7 +171,7 @@
       /**
        * Executes all the chained ImageOperations on the file
        * specified.
  -     * @param file The file to be processed
  +     * @param file The file to be processed.
        */
       public void processFile(File file) {
           try {
  @@ -190,26 +194,20 @@
               } else if (str_encoding.toLowerCase().equals("tif")) {
                   str_encoding = "TIFF";
               }
  -
               if (destDir == null) {
                   destDir = srcDir;
               }
  -
  -            File new_file = new File(destDir.getAbsolutePath()
  -                                     + File.separator + file.getName());
  +            File new_file = new File(destDir, file.getName());
   
               if ((overwrite && new_file.exists()) && 
(!new_file.equals(file))) {
                   new_file.delete();
               }
  -
               FileOutputStream stream = new FileOutputStream(new_file);
   
               JAI.create("encode", image, stream, str_encoding.toUpperCase(),
                          null);
               stream.flush();
               stream.close();
  -
  -
           } catch (IOException err) {
               if (!failonerror) {
                   log("Error processing file:  " + err);
  @@ -223,11 +221,11 @@
                   throw new BuildException(rerr);
               }
           }
  -
       }
   
       /**
  -     * Executes the Task
  +     * Executes the Task.
  +     * @throws BuildException on error.
        */
       public void execute() throws BuildException {
   
  @@ -238,15 +236,13 @@
               String[] files = null;
               ArrayList filesList = new ArrayList();
   
  -
               // deal with specified srcDir
               if (srcDir != null) {
                   ds = super.getDirectoryScanner(srcDir);
   
                   files = ds.getIncludedFiles();
                   for (int i = 0; i < files.length; i++) {
  -                    filesList.add(new File(srcDir.getAbsolutePath()
  -                                           + File.separator + files[i]));
  +                    filesList.add(new File(srcDir, files[i]));
                   }
               }
               // deal with the filesets
  @@ -256,26 +252,21 @@
                   files = ds.getIncludedFiles();
                   File fromDir = fs.getDir(getProject());
                   for (int j = 0; j < files.length; j++) {
  -                    filesList.add(new File(fromDir.getAbsolutePath()
  -                                           + File.separator + files[j]));
  +                    filesList.add(new File(fromDir, files[j]));
                   }
               }
  -
               if (!overwrite) {
                   // remove any files that shouldn't be overwritten.
                   ArrayList filesToRemove = new ArrayList();
                   for (Iterator i = filesList.iterator(); i.hasNext();) {
                       File f = (File) i.next();
  -                    File new_file = new File(destDir.getAbsolutePath()
  -                                             + File.separator + f.getName());
  +                    File new_file = new File(destDir, f.getName());
                       if (new_file.exists()) {
                           filesToRemove.add(f);
                       }
                   }
                   filesList.removeAll(filesToRemove);
               }
  -
  -
               // iterator through all the files and process them.
               for (Iterator i = filesList.iterator(); i.hasNext();) {
                   File file = (File) i.next();
  @@ -285,30 +276,26 @@
                       System.gc();
                   }
               }
  -
  -
           } catch (Exception err) {
               err.printStackTrace();
               throw new BuildException(err.getMessage());
           }
       }
   
  -
       /**
        * Ensure we have a consistent and legal set of attributes, and set
        * any internal flags necessary based on different combinations
        * of attributes.
  +     * @throws BuildException on error.
        */
       protected void validateAttributes() throws BuildException {
           if (srcDir == null && filesets.size() == 0) {
  -            throw new BuildException("Specify at least one source "
  -                                     + "- a srcDir or a fileset.");
  +            throw new BuildException("Specify at least one source"
  +                                     + "--a srcDir or a fileset.");
           }
  -
           if (srcDir == null && destDir == null) {
               throw new BuildException("Specify the destDir, or the srcDir.");
           }
       }
  -
   }
   
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to