ContainsRegexpSelector instance.
+ */
public ContainsRegexpSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer(
"{containsregexpselector expression: ");
@@ -162,7 +169,7 @@ public class ContainsRegexpSelector extends BaseExtendSelector {
while (teststr != null) {
- if (myExpression.matches(teststr) == true) {
+ if (myExpression.matches(teststr)) {
return true;
}
teststr = in.readLine();
@@ -176,7 +183,7 @@ public class ContainsRegexpSelector extends BaseExtendSelector {
try {
in.close();
} catch (Exception e) {
- throw new BuildException("Could not close file "
+ throw new BuildException("Could not close file "
+ filename);
}
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/ContainsSelector.java b/src/main/org/apache/tools/ant/types/selectors/ContainsSelector.java
index 1b12b5ffd..03b0c61e1 100644
--- a/src/main/org/apache/tools/ant/types/selectors/ContainsSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/ContainsSelector.java
@@ -76,14 +76,26 @@ public class ContainsSelector extends BaseExtendSelector {
private String contains = null;
private boolean casesensitive = true;
private boolean ignorewhitespace = false;
- public final static String CONTAINS_KEY = "text";
- public final static String CASE_KEY = "casesensitive";
- public final static String WHITESPACE_KEY = "ignorewhitespace";
+ /** Key to used for parameterized custom selector */
+ public static final String EXPRESSION_KEY = "expression";
+ /** Used for parameterized custom selector */
+ public static final String CONTAINS_KEY = "text";
+ /** Used for parameterized custom selector */
+ public static final String CASE_KEY = "casesensitive";
+ /** Used for parameterized custom selector */
+ public static final String WHITESPACE_KEY = "ignorewhitespace";
+ /**
+ * Creates a new ContainsSelector instance.
+ *
+ */
public ContainsSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{containsselector text: ");
buf.append(contains);
@@ -218,7 +230,7 @@ public class ContainsSelector extends BaseExtendSelector {
try {
in.close();
} catch (Exception e) {
- throw new BuildException("Could not close file "
+ throw new BuildException("Could not close file "
+ filename);
}
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/DateSelector.java b/src/main/org/apache/tools/ant/types/selectors/DateSelector.java
index c69ec479f..e151d969d 100644
--- a/src/main/org/apache/tools/ant/types/selectors/DateSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/DateSelector.java
@@ -77,18 +77,30 @@ public class DateSelector extends BaseExtendSelector {
private boolean includeDirs = false;
private int granularity = 0;
private int cmp = 2;
- public final static String MILLIS_KEY = "millis";
- public final static String DATETIME_KEY = "datetime";
- public final static String CHECKDIRS_KEY = "checkdirs";
- public final static String GRANULARITY_KEY = "granularity";
- public final static String WHEN_KEY = "when";
+ /** Key to used for parameterized custom selector */
+ public static final String MILLIS_KEY = "millis";
+ /** Key to used for parameterized custom selector */
+ public static final String DATETIME_KEY = "datetime";
+ /** Key to used for parameterized custom selector */
+ public static final String CHECKDIRS_KEY = "checkdirs";
+ /** Key to used for parameterized custom selector */
+ public static final String GRANULARITY_KEY = "granularity";
+ /** Key to used for parameterized custom selector */
+ public static final String WHEN_KEY = "when";
+ /**
+ * Creates a new DateSelector instance.
+ *
+ */
public DateSelector() {
if (Os.isFamily("dos")) {
granularity = 2000;
}
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{dateselector date: ");
buf.append(dateTime);
@@ -118,6 +130,7 @@ public class DateSelector extends BaseExtendSelector {
/**
* Returns the millisecond value the selector is set for.
+ * @return the millisecond value
*/
public long getMillis() {
return millis;
@@ -163,6 +176,7 @@ public class DateSelector extends BaseExtendSelector {
/**
* Sets the number of milliseconds leeway we will give before we consider
* a file not to have matched a date.
+ * @param granularity the number of milliconds leeway
*/
public void setGranularity(int granularity) {
this.granularity = granularity;
@@ -246,7 +260,7 @@ public class DateSelector extends BaseExtendSelector {
*/
public boolean isSelected(File basedir, String filename, File file) {
validate();
- if (file.isDirectory() && (includeDirs == false)) {
+ if (file.isDirectory() && (!includeDirs)) {
return true;
}
if (cmp == 0) {
@@ -263,6 +277,9 @@ public class DateSelector extends BaseExtendSelector {
*
*/
public static class TimeComparisons extends EnumeratedAttribute {
+ /**
+ * @return the values as an array of strings
+ */
public String[] getValues() {
return new String[]{"before", "after", "equal"};
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/DependSelector.java b/src/main/org/apache/tools/ant/types/selectors/DependSelector.java
index f63e9dce8..ca0012e8a 100644
--- a/src/main/org/apache/tools/ant/types/selectors/DependSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/DependSelector.java
@@ -67,10 +67,17 @@ import java.io.File;
*/
public class DependSelector extends MappingSelector {
+ /**
+ * Creates a new DependSelector instance.
+ *
+ */
public DependSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{dependselector targetdir: ");
if (targetdir == null) {
@@ -94,9 +101,9 @@ public class DependSelector extends MappingSelector {
/**
* this test is our selection test that compared the file with the destfile
- * @param srcfile
- * @param destfile
- * @return
+ * @param srcfile the source file
+ * @param destfile the destination file
+ * @return true if destination is out of date
*/
public boolean selectionTest(File srcfile, File destfile) {
boolean selected = SelectorUtils.isOutOfDate(srcfile, destfile,
diff --git a/src/main/org/apache/tools/ant/types/selectors/DepthSelector.java b/src/main/org/apache/tools/ant/types/selectors/DepthSelector.java
index eb26d9931..9003e1408 100644
--- a/src/main/org/apache/tools/ant/types/selectors/DepthSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/DepthSelector.java
@@ -71,12 +71,21 @@ public class DepthSelector extends BaseExtendSelector {
public int min = -1;
public int max = -1;
- public final static String MIN_KEY = "min";
- public final static String MAX_KEY = "max";
+ /** Used for parameterized custom selector */
+ public static final String MIN_KEY = "min";
+ /** Used for parameterized custom selector */
+ public static final String MAX_KEY = "max";
+ /**
+ * Creates a new DepthSelector instance.
+ *
+ */
public DepthSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{depthselector min: ");
buf.append(min);
@@ -169,20 +178,20 @@ public class DepthSelector extends BaseExtendSelector {
int depth = -1;
// If you felt daring, you could cache the basedir absolute path
- String abs_base = basedir.getAbsolutePath();
- String abs_file = file.getAbsolutePath();
- StringTokenizer tok_base = new StringTokenizer(abs_base,
+ String absBase = basedir.getAbsolutePath();
+ String absFile = file.getAbsolutePath();
+ StringTokenizer tokBase = new StringTokenizer(absBase,
File.separator);
- StringTokenizer tok_file = new StringTokenizer(abs_file,
+ StringTokenizer tokFile = new StringTokenizer(absFile,
File.separator);
- while (tok_file.hasMoreTokens()) {
- String filetoken = tok_file.nextToken();
- if (tok_base.hasMoreTokens()) {
- String basetoken = tok_base.nextToken();
+ while (tokFile.hasMoreTokens()) {
+ String filetoken = tokFile.nextToken();
+ if (tokBase.hasMoreTokens()) {
+ String basetoken = tokBase.nextToken();
// Sanity check. Ditch it if you want faster performance
if (!basetoken.equals(filetoken)) {
throw new BuildException("File " + filename
- + " does not appear within " + abs_base
+ + " does not appear within " + absBase
+ "directory");
}
} else {
@@ -192,9 +201,9 @@ public class DepthSelector extends BaseExtendSelector {
}
}
}
- if (tok_base.hasMoreTokens()) {
+ if (tokBase.hasMoreTokens()) {
throw new BuildException("File " + filename
- + " is outside of " + abs_base + "directory tree");
+ + " is outside of " + absBase + "directory tree");
}
if (min > -1 && depth < min) {
return false;
diff --git a/src/main/org/apache/tools/ant/types/selectors/DifferentSelector.java b/src/main/org/apache/tools/ant/types/selectors/DifferentSelector.java
index 232690d00..760cc8443 100644
--- a/src/main/org/apache/tools/ant/types/selectors/DifferentSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/DifferentSelector.java
@@ -78,6 +78,7 @@ import java.io.IOException;
* over as few files as possible, perhaps following it with an <uptodate;>
* to keep the descendent routines conditional.
*
+ * @author not specified
*/
public class DifferentSelector extends MappingSelector {
@@ -88,7 +89,7 @@ public class DifferentSelector extends MappingSelector {
/**
* This flag tells the selector to ignore file times in the comparison
- * @param ignoreFileTimes
+ * @param ignoreFileTimes if true ignore file times
*/
public void setIgnoreFileTimes(boolean ignoreFileTimes) {
this.ignoreFileTimes = ignoreFileTimes;
@@ -96,9 +97,9 @@ public class DifferentSelector extends MappingSelector {
/**
* this test is our selection test that compared the file with the destfile
- * @param srcfile
- * @param destfile
- * @return
+ * @param srcfile the source file
+ * @param destfile the destination file
+ * @return true if the files are different
*/
protected boolean selectionTest(File srcfile, File destfile) {
diff --git a/src/main/org/apache/tools/ant/types/selectors/ExtendSelector.java b/src/main/org/apache/tools/ant/types/selectors/ExtendSelector.java
index ed55de4b9..68d23eedf 100644
--- a/src/main/org/apache/tools/ant/types/selectors/ExtendSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/ExtendSelector.java
@@ -138,6 +138,7 @@ public class ExtendSelector extends BaseSelector {
/**
* Set the classpath to load the classname specified using an attribute.
+ * @param classpath the classpath to use
*/
public final void setClasspath(Path classpath) {
if (isReference()) {
@@ -152,6 +153,7 @@ public class ExtendSelector extends BaseSelector {
/**
* Specify the classpath to use to load the Selector (nested element).
+ * @return a classpath to be configured
*/
public final Path createClasspath() {
if (isReference()) {
@@ -165,6 +167,7 @@ public class ExtendSelector extends BaseSelector {
/**
* Get the classpath
+ * @return the classpath
*/
public final Path getClasspath() {
return classpath;
@@ -173,6 +176,7 @@ public class ExtendSelector extends BaseSelector {
/**
* Set the classpath to use for loading a custom selector by using
* a reference.
+ * @param r a reference to the classpath
*/
public void setClasspathref(Reference r) {
if (isReference()) {
@@ -211,6 +215,8 @@ public class ExtendSelector extends BaseSelector {
* since we know we must have them all by now. And since we must know
* both classpath and classname, creating the class is deferred to here
* as well.
+ *
+ * @exception BuildException if an error occurs
*/
public boolean isSelected(File basedir, String filename, File file)
throws BuildException {
diff --git a/src/main/org/apache/tools/ant/types/selectors/FilenameSelector.java b/src/main/org/apache/tools/ant/types/selectors/FilenameSelector.java
index 7a5a0dccc..3573f2c9d 100644
--- a/src/main/org/apache/tools/ant/types/selectors/FilenameSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/FilenameSelector.java
@@ -69,14 +69,25 @@ public class FilenameSelector extends BaseExtendSelector {
private String pattern = null;
private boolean casesensitive = true;
+
private boolean negated = false;
- public final static String NAME_KEY = "name";
- public final static String CASE_KEY = "casesensitive";
- public final static String NEGATE_KEY = "negate";
+ /** Used for parameterized custom selector */
+ public static final String NAME_KEY = "name";
+ /** Used for parameterized custom selector */
+ public static final String CASE_KEY = "casesensitive";
+ /** Used for parameterized custom selector */
+ public static final String NEGATE_KEY = "negate";
+ /**
+ * Creates a new FilenameSelector instance.
+ *
+ */
public FilenameSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{filenameselector name: ");
buf.append(pattern);
diff --git a/src/main/org/apache/tools/ant/types/selectors/MajoritySelector.java b/src/main/org/apache/tools/ant/types/selectors/MajoritySelector.java
index 6eafe33e4..f9e7b2ab0 100644
--- a/src/main/org/apache/tools/ant/types/selectors/MajoritySelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/MajoritySelector.java
@@ -79,6 +79,9 @@ public class MajoritySelector extends BaseSelectorContainer {
public MajoritySelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer();
if (hasSelectors()) {
@@ -89,6 +92,13 @@ public class MajoritySelector extends BaseSelectorContainer {
return buf.toString();
}
+ /**
+ * A attribute to specify what will happen if number
+ * of yes votes is the same as the number of no votes
+ * defaults to true
+ *
+ * @param tiebreaker the value to give if there is a tie
+ */
public void setAllowtie(boolean tiebreaker) {
allowtie = tiebreaker;
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/MappingSelector.java b/src/main/org/apache/tools/ant/types/selectors/MappingSelector.java
index 393227d22..aaba589ea 100644
--- a/src/main/org/apache/tools/ant/types/selectors/MappingSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/MappingSelector.java
@@ -65,6 +65,7 @@ import java.io.File;
/**
* A mapping selector is an abstract class adding mapping support to the base
* selector
+ * @author not specified
*/
public abstract class MappingSelector extends BaseSelector {
protected File targetdir = null;
@@ -72,6 +73,10 @@ public abstract class MappingSelector extends BaseSelector {
protected FileNameMapper map = null;
protected int granularity = 0;
+ /**
+ * Creates a new MappingSelector instance.
+ *
+ */
public MappingSelector() {
if (Os.isFamily("dos")) {
granularity = 2000;
@@ -91,6 +96,8 @@ public abstract class MappingSelector extends BaseSelector {
/**
* Defines the FileNameMapper to use (nested mapper element).
+ * @return a mapper to be configured
+ * @throws BuildException if more that one mapper defined
*/
public Mapper createMapper() throws BuildException {
if (mapperElement != null) {
@@ -155,7 +162,7 @@ public abstract class MappingSelector extends BaseSelector {
* this test is our selection test that compared the file with the destfile
* @param srcfile file to test; may be null
* @param destfile destination file
- * @return
+ * @return true if source file compares with destination file
*/
protected abstract boolean selectionTest(File srcfile, File destfile);
@@ -163,6 +170,7 @@ public abstract class MappingSelector extends BaseSelector {
* Sets the number of milliseconds leeway we will give before we consider
* a file out of date. Defaults to 2000 on MS-DOS derivatives as the FAT
* file system.
+ * @param granularity the leeway in milliseconds
*/
public void setGranularity(int granularity) {
this.granularity = granularity;
diff --git a/src/main/org/apache/tools/ant/types/selectors/PresentSelector.java b/src/main/org/apache/tools/ant/types/selectors/PresentSelector.java
index 335dd801a..3ef98b61d 100644
--- a/src/main/org/apache/tools/ant/types/selectors/PresentSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/PresentSelector.java
@@ -78,9 +78,16 @@ public class PresentSelector extends BaseSelector {
private FileNameMapper map = null;
private boolean destmustexist = true;
+ /**
+ * Creates a new PresentSelector instance.
+ *
+ */
public PresentSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{presentselector targetdir: ");
if (targetdir == null) {
@@ -115,6 +122,8 @@ public class PresentSelector extends BaseSelector {
/**
* Defines the FileNameMapper to use (nested mapper element).
+ * @return a mapper to be configured
+ * @throws BuildException if more that one mapper defined
*/
public Mapper createMapper() throws BuildException {
if (mapperElement != null) {
@@ -197,6 +206,9 @@ public class PresentSelector extends BaseSelector {
* presence is allowed and required.
*/
public static class FilePresence extends EnumeratedAttribute {
+ /**
+ * @return the values as an array of strings
+ */
public String[] getValues() {
return new String[]{"srconly", "both"};
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/SelectSelector.java b/src/main/org/apache/tools/ant/types/selectors/SelectSelector.java
index e3dcca8dc..6a8657cbb 100644
--- a/src/main/org/apache/tools/ant/types/selectors/SelectSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/SelectSelector.java
@@ -81,6 +81,9 @@ public class SelectSelector extends BaseSelectorContainer {
public SelectSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer();
if (hasSelectors()) {
@@ -111,6 +114,7 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Indicates whether there are any selectors here.
+ * @return whether any selectors are in this container
*/
public boolean hasSelectors() {
if (isReference()) {
@@ -121,6 +125,7 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Gives the count of the number of selectors in this container
+ * @return the number of selectors in this container
*/
public int selectorCount() {
if (isReference()) {
@@ -131,6 +136,8 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Returns the set of selectors as an array.
+ * @param p the current project
+ * @return an array of selectors in this container
*/
public FileSelector[] getSelectors(Project p) {
if (isReference()) {
@@ -141,6 +148,7 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Returns an enumerator for accessing the set of selectors.
+ * @return an enumerator that goes through each of the selectors
*/
public Enumeration selectorElements() {
if (isReference()) {
@@ -177,6 +185,7 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Ensures that the selector passes the conditions placed
* on it with if and unless.
+ * @return true if conditions are passed
*/
public boolean passesConditions() {
if (ifProperty != null
@@ -192,6 +201,7 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Sets the if attribute to a property which must exist for the
* selector to select any files.
+ * @param ifProperty the property to check
*/
public void setIf(String ifProperty) {
this.ifProperty = ifProperty;
@@ -200,6 +210,7 @@ public class SelectSelector extends BaseSelectorContainer {
/**
* Sets the unless attribute to a property which cannot exist for the
* selector to select any files.
+ * @param unlessProperty the property to check
*/
public void setUnless(String unlessProperty) {
this.unlessProperty = unlessProperty;
diff --git a/src/main/org/apache/tools/ant/types/selectors/SelectorContainer.java b/src/main/org/apache/tools/ant/types/selectors/SelectorContainer.java
index 515eb1f88..bccbf6c0b 100644
--- a/src/main/org/apache/tools/ant/types/selectors/SelectorContainer.java
+++ b/src/main/org/apache/tools/ant/types/selectors/SelectorContainer.java
@@ -71,129 +71,147 @@ public interface SelectorContainer {
*
* @return whether any selectors are in this container
*/
- public boolean hasSelectors();
+ boolean hasSelectors();
/**
* Gives the count of the number of selectors in this container
*
* @return the number of selectors in this container
*/
- public int selectorCount();
+ int selectorCount();
/**
* Returns the set of selectors as an array.
- *
+ * @param p the current project
* @return an array of selectors in this container
*/
- public FileSelector[] getSelectors(Project p);
+ FileSelector[] getSelectors(Project p);
/**
* Returns an enumerator for accessing the set of selectors.
*
* @return an enumerator that goes through each of the selectors
*/
- public Enumeration selectorElements();
+ Enumeration selectorElements();
/**
* Add a new selector into this container.
*
* @param selector the new selector to add
*/
- public void appendSelector(FileSelector selector);
+ void appendSelector(FileSelector selector);
/* Methods below all add specific selectors */
/**
* add a "Select" selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addSelector(SelectSelector selector);
+ void addSelector(SelectSelector selector);
/**
* add an "And" selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addAnd(AndSelector selector);
+ void addAnd(AndSelector selector);
/**
* add an "Or" selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addOr(OrSelector selector);
+ void addOr(OrSelector selector);
/**
* add a "Not" selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addNot(NotSelector selector);
+ void addNot(NotSelector selector);
/**
* add a "None" selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addNone(NoneSelector selector);
+ void addNone(NoneSelector selector);
/**
* add a majority selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addMajority(MajoritySelector selector);
+ void addMajority(MajoritySelector selector);
/**
* add a selector date entry on the selector list
+ * @param selector the selector to add
*/
- public void addDate(DateSelector selector);
+ void addDate(DateSelector selector);
/**
* add a selector size entry on the selector list
+ * @param selector the selector to add
*/
- public void addSize(SizeSelector selector);
+ void addSize(SizeSelector selector);
/**
* add a selector filename entry on the selector list
+ * @param selector the selector to add
*/
- public void addFilename(FilenameSelector selector);
+ void addFilename(FilenameSelector selector);
/**
* add an extended selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addCustom(ExtendSelector selector);
+ void addCustom(ExtendSelector selector);
/**
* add a contains selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addContains(ContainsSelector selector);
+ void addContains(ContainsSelector selector);
/**
* add a present selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addPresent(PresentSelector selector);
+ void addPresent(PresentSelector selector);
/**
* add a depth selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addDepth(DepthSelector selector);
+ void addDepth(DepthSelector selector);
/**
* add a depends selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addDepend(DependSelector selector);
+ void addDepend(DependSelector selector);
/**
* add a regular expression selector entry on the selector list
+ * @param selector the selector to add
*/
- public void addContainsRegexp(ContainsRegexpSelector selector);
+ void addContainsRegexp(ContainsRegexpSelector selector);
/**
* add the type selector
+ * @param selector the selector to add
* @since ant 1.6
*/
- public void addType(TypeSelector selector);
+ void addType(TypeSelector selector);
/**
* add the different selector
+ * @param selector the selector to add
* @since ant 1.6
*/
- public void addDifferent(DifferentSelector selector);
+ void addDifferent(DifferentSelector selector);
/**
* add an arbitary selector
+ * @param selector the selector to add
* @since Ant 1.6
*/
- public void add(FileSelector selector);
+ void add(FileSelector selector);
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/SelectorScanner.java b/src/main/org/apache/tools/ant/types/selectors/SelectorScanner.java
index f3c1ef817..30d388674 100644
--- a/src/main/org/apache/tools/ant/types/selectors/SelectorScanner.java
+++ b/src/main/org/apache/tools/ant/types/selectors/SelectorScanner.java
@@ -73,14 +73,14 @@ public interface SelectorScanner {
*
* @return list of directories not selected
*/
- public String[] getDeselectedDirectories();
+ String[] getDeselectedDirectories();
/**
* Files which were selected out of a scan.
*
* @return list of files not selected
*/
- public String[] getDeselectedFiles();
+ String[] getDeselectedFiles();
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java b/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
index 76a9e9a58..4f60a0b26 100644
--- a/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
+++ b/src/main/org/apache/tools/ant/types/selectors/SelectorUtils.java
@@ -674,7 +674,7 @@ public final class SelectorUtils {
/**
* removes from a pattern all tokens to the right containing wildcards
- * @param input
+ * @param input the input string
* @return the leftmost part of the pattern without wildcards
*/
public static String rtrimWildcardTokens(String input) {
diff --git a/src/main/org/apache/tools/ant/types/selectors/SizeSelector.java b/src/main/org/apache/tools/ant/types/selectors/SizeSelector.java
index 64f5d96c0..c28c43eff 100644
--- a/src/main/org/apache/tools/ant/types/selectors/SizeSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/SizeSelector.java
@@ -71,13 +71,23 @@ public class SizeSelector extends BaseExtendSelector {
private long multiplier = 1;
private long sizelimit = -1;
private int cmp = 2;
- public final static String SIZE_KEY = "value";
- public final static String UNITS_KEY = "units";
- public final static String WHEN_KEY = "when";
+ /** Used for parameterized custom selector */
+ public static final String SIZE_KEY = "value";
+ /** Used for parameterized custom selector */
+ public static final String UNITS_KEY = "units";
+ /** Used for parameterized custom selector */
+ public static final String WHEN_KEY = "when";
+ /**
+ * Creates a new SizeSelector instance.
+ *
+ */
public SizeSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{sizeselector value: ");
buf.append(sizelimit);
@@ -270,6 +280,9 @@ public class SizeSelector extends BaseExtendSelector {
* standard. It disambiguates things for us, though.
*/
public static class ByteUnits extends EnumeratedAttribute {
+ /**
+ * @return the values as an array of strings
+ */
public String[] getValues() {
return new String[]{"K", "k", "kilo", "KILO",
"Ki", "KI", "ki", "kibi", "KIBI",
@@ -287,6 +300,9 @@ public class SizeSelector extends BaseExtendSelector {
* Enumerated attribute with the values for size comparison.
*/
public static class SizeComparisons extends EnumeratedAttribute {
+ /**
+ * @return the values as an array of strings
+ */
public String[] getValues() {
return new String[]{"less", "more", "equal"};
}
diff --git a/src/main/org/apache/tools/ant/types/selectors/TypeSelector.java b/src/main/org/apache/tools/ant/types/selectors/TypeSelector.java
index 916cbc36f..2a5e2cf1d 100644
--- a/src/main/org/apache/tools/ant/types/selectors/TypeSelector.java
+++ b/src/main/org/apache/tools/ant/types/selectors/TypeSelector.java
@@ -69,11 +69,19 @@ public class TypeSelector extends BaseExtendSelector {
private String type = null;
- public final static String TYPE_KEY = "type";
+ /** Key to used for parameterized custom selector */
+ public static final String TYPE_KEY = "type";
+ /**
+ * Creates a new TypeSelector instance.
+ *
+ */
public TypeSelector() {
}
+ /**
+ * @return a string describing this object
+ */
public String toString() {
StringBuffer buf = new StringBuffer("{typeselector type: ");
buf.append(type);
@@ -83,6 +91,7 @@ public class TypeSelector extends BaseExtendSelector {
/**
* Set the type of file to require.
+ * @param fileTypes the type of file - file or dir
*/
public void setType(FileType fileTypes) {
this.type = fileTypes.getValue();
@@ -146,9 +155,14 @@ public class TypeSelector extends BaseExtendSelector {
* Enumerated attribute with the values for types of file
*/
public static class FileType extends EnumeratedAttribute {
+ /** the string value for file */
public static final String FILE = "file";
+ /** the string value for dir */
public static final String DIR = "dir";
+ /**
+ * @return the values as an array of strings
+ */
public String[] getValues() {
return new String[]{FILE, DIR};
}