Browse Source

Made loop counter a simple variable rather than a complex value

Fixing a audit violation


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@270921 13f79535-47bb-0310-9956-ffa450edef68
master
Peter Donald 23 years ago
parent
commit
076d8fda91
4 changed files with 22 additions and 16 deletions
  1. +2
    -1
      proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/file/Copy.java
  2. +9
    -7
      proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/file/Delete.java
  3. +2
    -1
      proposal/myrmidon/src/todo/org/apache/tools/ant/taskdefs/file/Copy.java
  4. +9
    -7
      proposal/myrmidon/src/todo/org/apache/tools/ant/taskdefs/file/Delete.java

+ 2
- 1
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/file/Copy.java View File

@@ -227,7 +227,8 @@ public class Copy
} }


// deal with the filesets // deal with the filesets
for( int i = 0; i < m_filesets.size(); i++ )
final int size = m_filesets.size();
for( int i = 0; i < size; i++ )
{ {
final FileSet fileSet = (FileSet)m_filesets.get( i ); final FileSet fileSet = (FileSet)m_filesets.get( i );
final DirectoryScanner scanner = ScannerUtil.getDirectoryScanner( fileSet ); final DirectoryScanner scanner = ScannerUtil.getDirectoryScanner( fileSet );


+ 9
- 7
proposal/myrmidon/src/main/org/apache/tools/ant/taskdefs/file/Delete.java View File

@@ -10,8 +10,8 @@ package org.apache.tools.ant.taskdefs.file;
import java.io.File; import java.io.File;
import java.util.ArrayList; import java.util.ArrayList;
import org.apache.myrmidon.api.TaskException; import org.apache.myrmidon.api.TaskException;
import org.apache.tools.ant.types.DirectoryScanner;
import org.apache.tools.ant.Task; import org.apache.tools.ant.Task;
import org.apache.tools.ant.types.DirectoryScanner;
import org.apache.tools.ant.types.FileSet; import org.apache.tools.ant.types.FileSet;
import org.apache.tools.ant.types.ScannerUtil; import org.apache.tools.ant.types.ScannerUtil;


@@ -122,15 +122,17 @@ public class Delete
} }


// delete the files in the filesets // delete the files in the filesets
for( int i = 0; i < filesets.size(); i++ )
final int size = filesets.size();
for( int i = 0; i < size; i++ )
{ {
FileSet fs = (FileSet)filesets.get( i );
final FileSet fileSet = (FileSet)filesets.get( i );
try try
{ {
DirectoryScanner ds = ScannerUtil.getDirectoryScanner( fs );
String[] files = ds.getIncludedFiles();
String[] dirs = ds.getIncludedDirectories();
removeFiles( fs.getDir(), files, dirs );
final DirectoryScanner scanner =
ScannerUtil.getDirectoryScanner( fileSet );
String[] files = scanner.getIncludedFiles();
String[] dirs = scanner.getIncludedDirectories();
removeFiles( fileSet.getDir(), files, dirs );
} }
catch( TaskException be ) catch( TaskException be )
{ {


+ 2
- 1
proposal/myrmidon/src/todo/org/apache/tools/ant/taskdefs/file/Copy.java View File

@@ -227,7 +227,8 @@ public class Copy
} }


// deal with the filesets // deal with the filesets
for( int i = 0; i < m_filesets.size(); i++ )
final int size = m_filesets.size();
for( int i = 0; i < size; i++ )
{ {
final FileSet fileSet = (FileSet)m_filesets.get( i ); final FileSet fileSet = (FileSet)m_filesets.get( i );
final DirectoryScanner scanner = ScannerUtil.getDirectoryScanner( fileSet ); final DirectoryScanner scanner = ScannerUtil.getDirectoryScanner( fileSet );


+ 9
- 7
proposal/myrmidon/src/todo/org/apache/tools/ant/taskdefs/file/Delete.java View File

@@ -10,8 +10,8 @@ package org.apache.tools.ant.taskdefs.file;
import java.io.File; import java.io.File;
import java.util.ArrayList; import java.util.ArrayList;
import org.apache.myrmidon.api.TaskException; import org.apache.myrmidon.api.TaskException;
import org.apache.tools.ant.types.DirectoryScanner;
import org.apache.tools.ant.Task; import org.apache.tools.ant.Task;
import org.apache.tools.ant.types.DirectoryScanner;
import org.apache.tools.ant.types.FileSet; import org.apache.tools.ant.types.FileSet;
import org.apache.tools.ant.types.ScannerUtil; import org.apache.tools.ant.types.ScannerUtil;


@@ -122,15 +122,17 @@ public class Delete
} }


// delete the files in the filesets // delete the files in the filesets
for( int i = 0; i < filesets.size(); i++ )
final int size = filesets.size();
for( int i = 0; i < size; i++ )
{ {
FileSet fs = (FileSet)filesets.get( i );
final FileSet fileSet = (FileSet)filesets.get( i );
try try
{ {
DirectoryScanner ds = ScannerUtil.getDirectoryScanner( fs );
String[] files = ds.getIncludedFiles();
String[] dirs = ds.getIncludedDirectories();
removeFiles( fs.getDir(), files, dirs );
final DirectoryScanner scanner =
ScannerUtil.getDirectoryScanner( fileSet );
String[] files = scanner.getIncludedFiles();
String[] dirs = scanner.getIncludedDirectories();
removeFiles( fileSet.getDir(), files, dirs );
} }
catch( TaskException be ) catch( TaskException be )
{ {


Loading…
Cancel
Save