donaldp     01/12/22 14:01:59

  Modified:    proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs
                        ExecuteOn.java Java.java Javac.java
               proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/exec
                        ExecTask.java
               proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional
                        ANTLR.java
               
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb
                        BorlandDeploymentTool.java DDCreator.java Ejbc.java
                        WebsphereDeploymentTool.java
               
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/javacc
                        JJTree.java JavaCC.java
               
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/jdepend
                        JDependTask.java
               
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/junit
                        JUnitTask.java
               
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/metamata
                        AbstractMetamataTask.java MParse.java
               
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/sitraka
                        Coverage.java
               proposal/myrmidon/src/main/org/apache/tools/ant/types
                        Commandline.java CommandlineJava.java
                        EnvironmentData.java
  Added:       proposal/myrmidon/src/main/org/apache/tools/ant/types
                        Argument.java Marker.java
  Log:
  Moved Commandline.Argument/Marker to top level classes.
  
  Revision  Changes    Path
  1.13      +5 -4      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/ExecuteOn.java
  
  Index: ExecuteOn.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/ExecuteOn.java,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -r1.12 -r1.13
  --- ExecuteOn.java    2001/12/22 12:59:30     1.12
  +++ ExecuteOn.java    2001/12/22 22:01:57     1.13
  @@ -20,6 +20,7 @@
   import org.apache.tools.ant.types.EnumeratedAttribute;
   import org.apache.tools.ant.types.FileSet;
   import org.apache.tools.ant.types.Mapper;
  +import org.apache.tools.ant.types.Marker;
   import org.apache.tools.ant.util.FileNameMapper;
   import org.apache.tools.ant.util.SourceFileScanner;
   
  @@ -36,9 +37,9 @@
       private boolean relative = false;
       private boolean parallel = false;
       protected String type = "file";
  -    protected Commandline.Marker srcFilePos = null;
  +    protected Marker srcFilePos = null;
       private boolean skipEmpty = false;
  -    protected Commandline.Marker targetFilePos = null;
  +    protected Marker targetFilePos = null;
       protected Mapper mapperElement = null;
       protected FileNameMapper mapper = null;
       protected File destDir = null;
  @@ -132,7 +133,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Marker createSrcfile()
  +    public Marker createSrcfile()
           throws TaskException
       {
           if( srcFilePos != null )
  @@ -149,7 +150,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Marker createTargetfile()
  +    public Marker createTargetfile()
           throws TaskException
       {
           if( targetFilePos != null )
  
  
  
  1.11      +3 -2      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/Java.java
  
  Index: Java.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/Java.java,v
  retrieving revision 1.10
  retrieving revision 1.11
  diff -u -r1.10 -r1.11
  --- Java.java 2001/12/22 21:46:12     1.10
  +++ Java.java 2001/12/22 22:01:57     1.11
  @@ -24,6 +24,7 @@
   import org.apache.tools.ant.types.Path;
   import org.apache.tools.ant.types.Reference;
   import org.apache.tools.ant.types.EnvironmentVariable;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * This task acts as a loader for java applications but allows to use the 
same
  @@ -185,7 +186,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createArg()
  +    public Argument createArg()
       {
           return cmdl.createArgument();
       }
  @@ -206,7 +207,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createJvmarg()
  +    public Argument createJvmarg()
       {
           return cmdl.createVmArgument();
       }
  
  
  
  1.11      +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/Javac.java
  
  Index: Javac.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/Javac.java,v
  retrieving revision 1.10
  retrieving revision 1.11
  diff -u -r1.10 -r1.11
  --- Javac.java        2001/12/22 12:59:30     1.10
  +++ Javac.java        2001/12/22 22:01:57     1.11
  @@ -19,6 +19,7 @@
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.Path;
   import org.apache.tools.ant.types.Reference;
  +import org.apache.tools.ant.types.Argument;
   import org.apache.tools.ant.util.GlobPatternMapper;
   import org.apache.tools.ant.util.SourceFileScanner;
   
  @@ -914,7 +915,7 @@
        * @author RT
        */
       public class ImplementationSpecificArgument
  -        extends Commandline.Argument
  +        extends Argument
       {
   
           private String impl;
  
  
  
  1.6       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/exec/ExecTask.java
  
  Index: ExecTask.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/exec/ExecTask.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- ExecTask.java     2001/12/22 21:46:12     1.5
  +++ ExecTask.java     2001/12/22 22:01:58     1.6
  @@ -22,6 +22,7 @@
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.EnvironmentData;
   import org.apache.tools.ant.types.EnvironmentVariable;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Executes a given command if the os platform is appropriate.
  @@ -186,7 +187,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createArg()
  +    public Argument createArg()
       {
           return cmdl.createArgument();
       }
  
  
  
  1.9       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ANTLR.java
  
  Index: ANTLR.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ANTLR.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- ANTLR.java        2001/12/22 12:57:36     1.8
  +++ ANTLR.java        2001/12/22 22:01:58     1.9
  @@ -21,6 +21,7 @@
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.CommandlineJava;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * ANTLR task.
  @@ -104,7 +105,7 @@
        *      the JVM.
        * @see #setFork(boolean)
        */
  -    public Commandline.Argument createJvmarg()
  +    public Argument createJvmarg()
       {
           return commandline.createVmArgument();
       }
  
  
  
  1.10      +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/BorlandDeploymentTool.java
  
  Index: BorlandDeploymentTool.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/BorlandDeploymentTool.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -r1.9 -r1.10
  --- BorlandDeploymentTool.java        2001/12/22 12:57:37     1.9
  +++ BorlandDeploymentTool.java        2001/12/22 22:01:58     1.10
  @@ -23,6 +23,7 @@
   import org.apache.tools.ant.taskdefs.exec.ExecuteStreamHandler;
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * BorlandDeploymentTool is dedicated to the Borland Application Server 4.5 
and
  @@ -491,7 +492,7 @@
   
               javaTask = (Java)getTask().getProject().createTask( "java" );
               javaTask.setClassname( VERIFY );
  -            Commandline.Argument arguments = javaTask.createArg();
  +            Argument arguments = javaTask.createArg();
               arguments.setLine( args );
               Path classpath = getCombinedClasspath();
               if( classpath != null )
  
  
  
  1.5       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/DDCreator.java
  
  Index: DDCreator.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/DDCreator.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- DDCreator.java    2001/12/21 13:34:17     1.4
  +++ DDCreator.java    2001/12/22 22:01:58     1.5
  @@ -14,6 +14,7 @@
   import org.apache.tools.ant.taskdefs.MatchingTask;
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Build a serialised deployment descriptor given a text file description of 
the
  @@ -122,7 +123,7 @@
           Java ddCreatorTask = (Java)getProject().createTask( "java" );
           ddCreatorTask.setFork( true );
           ddCreatorTask.setClassname( 
"org.apache.tools.ant.taskdefs.optional.ejb.DDCreatorHelper" );
  -        Commandline.Argument arguments = ddCreatorTask.createArg();
  +        Argument arguments = ddCreatorTask.createArg();
           arguments.setLine( args );
           ddCreatorTask.setClasspath( new Path( getProject(), execClassPath ) 
);
           if( ddCreatorTask.executeJava() != 0 )
  
  
  
  1.6       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/Ejbc.java
  
  Index: Ejbc.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/Ejbc.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- Ejbc.java 2001/12/22 12:57:37     1.5
  +++ Ejbc.java 2001/12/22 22:01:58     1.6
  @@ -14,6 +14,7 @@
   import org.apache.tools.ant.taskdefs.MatchingTask;
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Build EJB support classes using Weblogic's ejbc tool from a directory
  @@ -181,7 +182,7 @@
               args += " " + files[ i ];
           }
   
  -        Commandline.Argument arguments = helperTask.createArg();
  +        Argument arguments = helperTask.createArg();
           arguments.setLine( args );
           helperTask.setClasspath( new Path( getProject(), execClassPath ) );
           if( helperTask.executeJava() != 0 )
  
  
  
  1.8       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/WebsphereDeploymentTool.java
  
  Index: WebsphereDeploymentTool.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/ejb/WebsphereDeploymentTool.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- WebsphereDeploymentTool.java      2001/12/22 21:46:12     1.7
  +++ WebsphereDeploymentTool.java      2001/12/22 22:01:58     1.8
  @@ -25,6 +25,7 @@
   import org.apache.tools.ant.types.EnvironmentData;
   import org.apache.tools.ant.types.Path;
   import org.apache.tools.ant.types.EnvironmentVariable;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Websphere deployment tool that augments the ejbjar task.
  @@ -982,7 +983,7 @@
   
                   // Set the Java class name
                   javaTask.setClassname( "com.ibm.etools.ejbdeploy.EJBDeploy" 
);
  -                Commandline.Argument arguments = javaTask.createArg();
  +                Argument arguments = javaTask.createArg();
                   arguments.setLine( args );
                   Path classpath = wasClasspath;
                   if( classpath == null )
  
  
  
  1.9       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JJTree.java
  
  Index: JJTree.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JJTree.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- JJTree.java       2001/12/22 12:57:38     1.8
  +++ JJTree.java       2001/12/22 22:01:58     1.9
  @@ -19,6 +19,7 @@
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.CommandlineJava;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Taskdef for the JJTree compiler compiler.
  @@ -180,7 +181,7 @@
                                                  "/JavaCC.zip" );
           classpath.addJavaRuntime();
   
  -        final Commandline.Argument arg = cmdl.createVmArgument();
  +        final Argument arg = cmdl.createVmArgument();
           arg.setValue( "-mx140M" );
           arg.setValue( "-Dinstall.root=" + javaccHome.getAbsolutePath() );
   
  
  
  
  1.7       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JavaCC.java
  
  Index: JavaCC.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JavaCC.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- JavaCC.java       2001/12/22 12:57:38     1.6
  +++ JavaCC.java       2001/12/22 22:01:58     1.7
  @@ -17,6 +17,7 @@
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.CommandlineJava;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Taskdef for the JavaCC compiler compiler.
  @@ -230,7 +231,7 @@
                                                  "/JavaCC.zip" );
           classpath.addJavaRuntime();
   
  -        final Commandline.Argument arg = cmdl.createVmArgument();
  +        final Argument arg = cmdl.createVmArgument();
           arg.setValue( "-mx140M" );
           arg.setValue( "-Dinstall.root=" + javaccHome.getAbsolutePath() );
   
  
  
  
  1.8       +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java
  
  Index: JDependTask.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- JDependTask.java  2001/12/22 12:57:38     1.7
  +++ JDependTask.java  2001/12/22 22:01:58     1.8
  @@ -23,6 +23,7 @@
   import org.apache.tools.ant.types.EnumeratedAttribute;
   import org.apache.tools.ant.types.Path;
   import org.apache.tools.ant.types.Reference;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Ant task to run JDepend tests. <p>
  @@ -219,7 +220,7 @@
        *      the JVM.
        * @see #setFork(boolean)
        */
  -    public Commandline.Argument createJvmarg( CommandlineJava commandline )
  +    public Argument createJvmarg( CommandlineJava commandline )
       {
           return commandline.createVmArgument();
       }
  
  
  
  1.12      +2 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java
  
  Index: JUnitTask.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java,v
  retrieving revision 1.11
  retrieving revision 1.12
  diff -u -r1.11 -r1.12
  --- JUnitTask.java    2001/12/22 21:46:13     1.11
  +++ JUnitTask.java    2001/12/22 22:01:58     1.12
  @@ -31,6 +31,7 @@
   import org.apache.tools.ant.types.EnvironmentData;
   import org.apache.tools.ant.types.Path;
   import org.apache.tools.ant.types.EnvironmentVariable;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Ant task to run JUnit tests. <p>
  @@ -367,7 +368,7 @@
        *      the JVM.
        * @see #setFork(boolean)
        */
  -    public Commandline.Argument createJvmarg()
  +    public Argument createJvmarg()
       {
           return commandline.createVmArgument();
       }
  
  
  
  1.8       +4 -3      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/metamata/AbstractMetamataTask.java
  
  Index: AbstractMetamataTask.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/metamata/AbstractMetamataTask.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- AbstractMetamataTask.java 2001/12/22 12:57:38     1.7
  +++ AbstractMetamataTask.java 2001/12/22 22:01:58     1.8
  @@ -25,6 +25,7 @@
   import org.apache.tools.ant.types.CommandlineJava;
   import org.apache.tools.ant.types.FileSet;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Somewhat abstract framework to be used for other metama 2.0 tasks. This
  @@ -169,7 +170,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createJvmarg()
  +    public Argument createJvmarg()
       {
           return cmdl.createVmArgument();
       }
  @@ -226,7 +227,7 @@
           classPath.createPathElement().setLocation( jar );
   
           // set the metamata.home property
  -        final Commandline.Argument vmArgs = cmdl.createVmArgument();
  +        final Argument vmArgs = cmdl.createVmArgument();
           vmArgs.setValue( "-Dmetamata.home=" + metamataHome.getAbsolutePath() 
);
   
           // retrieve all the files we want to scan
  @@ -237,7 +238,7 @@
           ArrayList options = getOptions();
           optionsFile = createTmpFile();
           generateOptionsFile( optionsFile, options );
  -        Commandline.Argument args = cmdl.createArgument();
  +        Argument args = cmdl.createArgument();
           args.setLine( "-arguments " + optionsFile.getAbsolutePath() );
       }
   
  
  
  
  1.10      +4 -3      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/metamata/MParse.java
  
  Index: MParse.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/metamata/MParse.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -r1.9 -r1.10
  --- MParse.java       2001/12/22 12:57:38     1.9
  +++ MParse.java       2001/12/22 22:01:58     1.10
  @@ -22,6 +22,7 @@
   import org.apache.tools.ant.types.Commandline;
   import org.apache.tools.ant.types.CommandlineJava;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Simple Metamata MParse task based on the original written by <a
  @@ -160,7 +161,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createJvmarg()
  +    public Argument createJvmarg()
       {
           return cmdl.createVmArgument();
       }
  @@ -218,14 +219,14 @@
           }
   
           // set the metamata.home property
  -        final Commandline.Argument vmArgs = cmdl.createVmArgument();
  +        final Argument vmArgs = cmdl.createVmArgument();
           vmArgs.setValue( "-Dmetamata.home=" + metahome.getAbsolutePath() );
   
           // write all the options to a temp file and use it ro run the process
           String[] options = getOptions();
           optionsFile = createTmpFile();
           generateOptionsFile( optionsFile, options );
  -        Commandline.Argument args = cmdl.createArgument();
  +        Argument args = cmdl.createArgument();
           args.setLine( "-arguments " + optionsFile.getAbsolutePath() );
       }
   
  
  
  
  1.8       +3 -2      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/sitraka/Coverage.java
  
  Index: Coverage.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/optional/sitraka/Coverage.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- Coverage.java     2001/12/22 12:57:39     1.7
  +++ Coverage.java     2001/12/22 22:01:58     1.8
  @@ -25,6 +25,7 @@
   import org.apache.tools.ant.types.EnumeratedAttribute;
   import org.apache.tools.ant.types.FileSet;
   import org.apache.tools.ant.types.Path;
  +import org.apache.tools.ant.types.Argument;
   
   /**
    * Convenient task to run Sitraka JProbe Coverage from Ant. Options are 
pretty
  @@ -217,7 +218,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createArg()
  +    public Argument createArg()
       {
           return cmdlJava.createArgument();
       }
  @@ -244,7 +245,7 @@
        *
        * @return Description of the Returned Value
        */
  -    public Commandline.Argument createJvmarg()
  +    public Argument createJvmarg()
       {
           return cmdlJava.createVmArgument();
       }
  
  
  
  1.6       +11 -114   
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/Commandline.java
  
  Index: Commandline.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/Commandline.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- Commandline.java  2001/12/22 12:53:45     1.5
  +++ Commandline.java  2001/12/22 22:01:58     1.6
  @@ -35,13 +35,12 @@
   public class Commandline
       implements Cloneable
   {
  -    private ArrayList m_arguments = new ArrayList();
  +    protected final ArrayList m_arguments = new ArrayList();
       private String m_executable;
   
       public Commandline( String to_process )
           throws TaskException
       {
  -        super();
           String[] tmp = translateCommandline( to_process );
           if( tmp != null && tmp.length > 0 )
           {
  @@ -206,11 +205,11 @@
        *
        * @param executable The new Executable value
        */
  -    public void setExecutable( String executable )
  +    public void setExecutable( final String executable )
       {
           if( executable == null || executable.length() == 0 )
               return;
  -        this.m_executable = executable.replace( '/', File.separatorChar )
  +        m_executable = executable.replace( '/', File.separatorChar )
               .replace( '\\', File.separatorChar );
       }
   
  @@ -222,11 +221,12 @@
        */
       public String[] getArguments()
       {
  -        ArrayList result = new ArrayList( m_arguments.size() * 2 );
  -        for( int i = 0; i < m_arguments.size(); i++ )
  +        final int size = m_arguments.size();
  +        final ArrayList result = new ArrayList( size * 2 );
  +        for( int i = 0; i < size; i++ )
           {
  -            Argument arg = (Argument)m_arguments.get( i );
  -            String[] s = arg.getParts();
  +            final Argument arg = (Argument)m_arguments.get( i );
  +            final String[] s = arg.getParts();
               for( int j = 0; j < s.length; j++ )
               {
                   result.add( s[ j ] );
  @@ -246,7 +246,9 @@
       {
           final String[] args = getArguments();
           if( m_executable == null )
  +        {
               return args;
  +        }
           final String[] result = new String[ args.length + 1 ];
           result[ 0 ] = m_executable;
           System.arraycopy( args, 0, result, 1, args.length );
  @@ -315,7 +317,7 @@
        */
       public Marker createMarker()
       {
  -        return new Marker( m_arguments.size() );
  +        return new Marker( this, m_arguments.size() );
       }
   
       public int size()
  @@ -328,109 +330,4 @@
           return toString( getCommandline() );
       }
   
  -    /**
  -     * Used for nested xml command line definitions.
  -     *
  -     * @author RT
  -     */
  -    public static class Argument
  -    {
  -
  -        private String[] parts;
  -
  -        /**
  -         * Sets a single commandline argument to the absolute filename of the
  -         * given file.
  -         *
  -         * @param value a single commandline argument.
  -         */
  -        public void setFile( File value )
  -        {
  -            parts = new String[]{value.getAbsolutePath()};
  -        }
  -
  -        /**
  -         * Line to split into several commandline arguments.
  -         *
  -         * @param line line to split into several commandline arguments
  -         */
  -        public void setLine( String line )
  -            throws TaskException
  -        {
  -            parts = translateCommandline( line );
  -        }
  -
  -        /**
  -         * Sets a single commandline argument and treats it like a PATH -
  -         * ensures the right separator for the local platform is used.
  -         *
  -         * @param value a single commandline argument.
  -         */
  -        public void setPath( Path value )
  -        {
  -            parts = new String[]{value.toString()};
  -        }
  -
  -        /**
  -         * Sets a single commandline argument.
  -         *
  -         * @param value a single commandline argument.
  -         */
  -        public void setValue( String value )
  -        {
  -            parts = new String[]{value};
  -        }
  -
  -        /**
  -         * Returns the parts this Argument consists of.
  -         *
  -         * @return The Parts value
  -         */
  -        public String[] getParts()
  -        {
  -            return parts;
  -        }
  -    }
  -
  -    /**
  -     * Class to keep track of the position of an Argument.
  -     *
  -     * @author RT
  -     */
  -    // <p>This class is there to support the srcfile and targetfile
  -    // elements of &lt;execon&gt; and &lt;transform&gt; - don't know
  -    // whether there might be additional use cases.</p> --SB
  -    public class Marker
  -    {
  -        private int realPos = -1;
  -
  -        private int position;
  -
  -        Marker( int position )
  -        {
  -            this.position = position;
  -        }
  -
  -        /**
  -         * Return the number of arguments that preceeded this marker. <p>
  -         *
  -         * The name of the executable - if set - is counted as the very first
  -         * argument.</p>
  -         *
  -         * @return The Position value
  -         */
  -        public int getPosition()
  -        {
  -            if( realPos == -1 )
  -            {
  -                realPos = ( m_executable == null ? 0 : 1 );
  -                for( int i = 0; i < position; i++ )
  -                {
  -                    Argument arg = (Argument)m_arguments.get( i );
  -                    realPos += arg.getParts().length;
  -                }
  -            }
  -            return realPos;
  -        }
  -    }
   }
  
  
  
  1.7       +2 -2      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/CommandlineJava.java
  
  Index: CommandlineJava.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/CommandlineJava.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- CommandlineJava.java      2001/12/22 21:46:13     1.6
  +++ CommandlineJava.java      2001/12/22 22:01:58     1.7
  @@ -218,7 +218,7 @@
           return c;
       }
   
  -    public Commandline.Argument createArgument()
  +    public Argument createArgument()
       {
           return javaCommand.createArgument();
       }
  @@ -232,7 +232,7 @@
           return classpath;
       }
   
  -    public Commandline.Argument createVmArgument()
  +    public Argument createVmArgument()
       {
           return vmCommand.createArgument();
       }
  
  
  
  1.4       +0 -1      
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/EnvironmentData.java
  
  Index: EnvironmentData.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/EnvironmentData.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- EnvironmentData.java      2001/12/22 21:46:13     1.3
  +++ EnvironmentData.java      2001/12/22 22:01:58     1.4
  @@ -38,5 +38,4 @@
       {
           m_variables.add( var );
       }
  -
   }
  
  
  
  1.1                  
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/Argument.java
  
  Index: Argument.java
  ===================================================================
  /*
   * Copyright (C) The Apache Software Foundation. All rights reserved.
   *
   * This software is published under the terms of the Apache Software License
   * version 1.1, a copy of which has been included  with this distribution in
   * the LICENSE.txt file.
   */
  package org.apache.tools.ant.types;
  
  import java.io.File;
  import org.apache.myrmidon.api.TaskException;
  
  /**
   * Used for nested xml command line definitions.
   */
  public class Argument
  {
      private String[] m_parts;
  
      /**
       * Sets a single commandline argument to the absolute filename of the
       * given file.
       *
       * @param value a single commandline argument.
       */
      public void setFile( final File value )
      {
          m_parts = new String[]{value.getAbsolutePath()};
      }
  
      /**
       * Line to split into several commandline arguments.
       *
       * @param line line to split into several commandline arguments
       */
      public void setLine( final String line )
          throws TaskException
      {
          m_parts = Commandline.translateCommandline( line );
      }
  
      /**
       * Sets a single commandline argument and treats it like a PATH -
       * ensures the right separator for the local platform is used.
       *
       * @param value a single commandline argument.
       */
      public void setPath( final Path value )
      {
          m_parts = new String[]{value.toString()};
      }
  
      /**
       * Sets a single commandline argument.
       *
       * @param value a single commandline argument.
       */
      public void setValue( final String value )
      {
          m_parts = new String[]{value};
      }
  
      /**
       * Returns the parts this Argument consists of.
       *
       * @return The Parts value
       */
      public String[] getParts()
      {
          return m_parts;
      }
  }
  
  
  
  1.1                  
jakarta-ant/proposal/myrmidon/src/main/org/apache/tools/ant/types/Marker.java
  
  Index: Marker.java
  ===================================================================
  /*
   * Copyright (C) The Apache Software Foundation. All rights reserved.
   *
   * This software is published under the terms of the Apache Software License
   * version 1.1, a copy of which has been included  with this distribution in
   * the LICENSE.txt file.
   */
  package org.apache.tools.ant.types;
  
  import java.util.ArrayList;
  
  /**
   * Class to keep track of the position of an Argument.
   *
   * <p>This class is there to support the srcfile and targetfile
   * elements of &lt;execon&gt; and &lt;transform&gt; - don't know
   * whether there might be additional use cases.</p> --SB
   */
  public class Marker
  {
      private int m_realPos = -1;
      private int m_position;
      private Commandline m_commandline;
  
      Marker( Commandline commandline, int position )
      {
          m_commandline = commandline;
          m_position = position;
      }
  
      /**
       * Return the number of arguments that preceeded this marker. <p>
       *
       * The name of the executable - if set - is counted as the very first
       * argument.</p>
       *
       * @return The Position value
       */
      public int getPosition()
      {
          if( m_realPos == -1 )
          {
              m_realPos = ( m_commandline.getExecutable() == null ? 0 : 1 );
              final ArrayList arguments = m_commandline.m_arguments;
              for( int i = 0; i < m_position; i++ )
              {
                  final Argument arg = (Argument)arguments.get( i );
                  m_realPos += arg.getParts().length;
              }
          }
          return m_realPos;
      }
  }
  
  
  

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

Reply via email to