Author: peterreilly Date: Sun Nov 12 14:39:00 2006 New Revision: 474079 URL: http://svn.apache.org/viewvc?view=rev&rev=474079 Log: checkstyle
Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/Base64Converter.java ant/core/trunk/src/main/org/apache/tools/ant/util/ChainedMapper.java ant/core/trunk/src/main/org/apache/tools/ant/util/ClasspathUtils.java ant/core/trunk/src/main/org/apache/tools/ant/util/CompositeMapper.java ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatFileInputStream.java ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatResourceInputStream.java ant/core/trunk/src/main/org/apache/tools/ant/util/DOMElementWriter.java ant/core/trunk/src/main/org/apache/tools/ant/util/DOMUtils.java ant/core/trunk/src/main/org/apache/tools/ant/util/FileTokenizer.java ant/core/trunk/src/main/org/apache/tools/ant/util/FileUtils.java ant/core/trunk/src/main/org/apache/tools/ant/util/GlobPatternMapper.java ant/core/trunk/src/main/org/apache/tools/ant/util/IdentityStack.java ant/core/trunk/src/main/org/apache/tools/ant/util/JavaEnvUtils.java ant/core/trunk/src/main/org/apache/tools/ant/util/LazyFileOutputStream.java ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java ant/core/trunk/src/main/org/apache/tools/ant/util/LeadPipeInputStream.java ant/core/trunk/src/main/org/apache/tools/ant/util/LineOrientedOutputStream.java ant/core/trunk/src/main/org/apache/tools/ant/util/MergingMapper.java ant/core/trunk/src/main/org/apache/tools/ant/util/ProxySetup.java ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java ant/core/trunk/src/main/org/apache/tools/ant/util/RetryHandler.java ant/core/trunk/src/main/org/apache/tools/ant/util/Retryable.java ant/core/trunk/src/main/org/apache/tools/ant/util/SourceFileScanner.java ant/core/trunk/src/main/org/apache/tools/ant/util/StringTokenizer.java ant/core/trunk/src/main/org/apache/tools/ant/util/StringUtils.java ant/core/trunk/src/main/org/apache/tools/ant/util/UUEncoder.java ant/core/trunk/src/main/org/apache/tools/ant/util/WeakishReference.java ant/core/trunk/src/main/org/apache/tools/ant/util/XmlConstants.java ant/core/trunk/src/main/org/apache/tools/ant/util/java15/ProxyDiagnostics.java ant/core/trunk/src/main/org/apache/tools/ant/util/optional/ScriptRunner.java ant/core/trunk/src/main/org/apache/tools/ant/util/optional/WeakishReference12.java ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroMatcher.java ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/Base64Converter.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/Base64Converter.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/Base64Converter.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/Base64Converter.java Sun Nov 12 14:39:00 2006 @@ -25,7 +25,7 @@ **/ public class Base64Converter { - private final static char[] ALPHABET = { + private static final char[] ALPHABET = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', // 0 to 7 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', // 8 to 15 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', // 16 to 23 @@ -35,8 +35,10 @@ 'w', 'x', 'y', 'z', '0', '1', '2', '3', // 48 to 55 '4', '5', '6', '7', '8', '9', '+', '/'}; // 56 to 63 + // CheckStyle:ConstantNameCheck OFF - bc /** Provided for BC purposes */ public static final char[] alphabet = ALPHABET; + // CheckStyle:ConstantNameCheck ON /** Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/ChainedMapper.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/ChainedMapper.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/ChainedMapper.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/ChainedMapper.java Sun Nov 12 14:39:00 2006 @@ -31,7 +31,7 @@ */ public class ChainedMapper extends ContainerMapper { - //inherit doc + /** [EMAIL PROTECTED] */ public String[] mapFileName(String sourceFileName) { List inputs = new ArrayList(); List results = new ArrayList(); Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/ClasspathUtils.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/ClasspathUtils.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/ClasspathUtils.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/ClasspathUtils.java Sun Nov 12 14:39:00 2006 @@ -263,8 +263,7 @@ try { Class clazz = Class.forName(className, true, userDefinedLoader); Object o = clazz.newInstance(); - if (!expectedType.isInstance(o)) - { + if (!expectedType.isInstance(o)) { throw new BuildException( "Class of unexpected Type: " + className Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/CompositeMapper.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/CompositeMapper.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/CompositeMapper.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/CompositeMapper.java Sun Nov 12 14:39:00 2006 @@ -27,7 +27,7 @@ */ public class CompositeMapper extends ContainerMapper { - //inherit doc + /** [EMAIL PROTECTED] */ public String[] mapFileName(String sourceFileName) { HashSet results = new HashSet(); Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatFileInputStream.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatFileInputStream.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatFileInputStream.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatFileInputStream.java Sun Nov 12 14:39:00 2006 @@ -51,13 +51,20 @@ this.file = file; } - // inherit doc + /** + * Close the stream. + * @throws IOException if there is an error. + */ public void close() throws IOException { closeCurrent(); eof = true; } - // inherit doc + /** + * Read a byte. + * @return the byte (0 - 255) or -1 if this is the end of the stream. + * @throws IOException if there is an error. + */ public int read() throws IOException { int result = readCurrent(); if (result == EOF && !eof) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatResourceInputStream.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatResourceInputStream.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatResourceInputStream.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/ConcatResourceInputStream.java Sun Nov 12 14:39:00 2006 @@ -67,13 +67,20 @@ return ignoreErrors; } - // inherit doc - public void close() throws IOException { + /** + * Close the stream. + * @throws IOException if there is an error. + */ + public void close() throws IOException { closeCurrent(); eof = true; } - // inherit doc + /** + * Read a byte. + * @return the byte (0 - 255) or -1 if this is the end of the stream. + * @throws IOException if there is an error. + */ public int read() throws IOException { if (eof) { return EOF; Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/DOMElementWriter.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/DOMElementWriter.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/DOMElementWriter.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/DOMElementWriter.java Sun Nov 12 14:39:00 2006 @@ -130,6 +130,7 @@ * XML namespaces will be ignored. * @param xmlDeclaration flag to indicate whether the ?xml? declaration * should be included. + * @param namespacePolicy the policy to use. * @since Ant1.7 */ public DOMElementWriter(boolean xmlDeclaration, @@ -140,11 +141,13 @@ private static String lSep = System.getProperty("line.separator"); + // CheckStyle:VisibilityModifier OFF - bc /** * Don't try to be too smart but at least recognize the predefined * entities. */ protected String[] knownEntities = {"gt", "amp", "lt", "apos", "quot"}; + // CheckStyle:VisibilityModifier ON /** @@ -164,7 +167,9 @@ } /** - * Writes the XML declaration. + * Writes the XML declaration if xmlDeclaration is true. + * @param wri the writer to write to. + * @throws IOException if there is an error. * @since Ant 1.7.0 */ public void writeXMLDeclaration(Writer wri) throws IOException { @@ -365,6 +370,7 @@ * @param indent number of * @param indentWith string that should be used to indent the * corresponding tag. + * @param hasChildren if true indent. * @throws IOException if an error happens while writing to the stream. */ public void closeElement(Element element, Writer out, int indent, Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/DOMUtils.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/DOMUtils.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/DOMUtils.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/DOMUtils.java Sun Nov 12 14:39:00 2006 @@ -35,7 +35,7 @@ /** * Get a new Document instance, - * + * @return the document. * @since Ant 1.6.3 */ public static Document newDocument() { @@ -163,4 +163,4 @@ Element e = createChildElement(parent, name); appendCDATA(e, content); } -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/FileTokenizer.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/FileTokenizer.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/FileTokenizer.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/FileTokenizer.java Sun Nov 12 14:39:00 2006 @@ -20,8 +20,6 @@ import java.io.IOException; import java.io.Reader; import org.apache.tools.ant.ProjectComponent; -import org.apache.tools.ant.util.FileUtils; -import org.apache.tools.ant.util.Tokenizer; /** * Class to read the complete input into a string. Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/FileUtils.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/FileUtils.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/FileUtils.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/FileUtils.java Sun Nov 12 14:39:00 2006 @@ -1105,7 +1105,8 @@ File f = new File(path).getAbsoluteFile(); java.lang.reflect.Method toURIMethod = File.class.getMethod("toURI", new Class[0]); Object uriObj = toURIMethod.invoke(f, new Object[] {}); - java.lang.reflect.Method toASCIIStringMethod = uriClazz.getMethod("toASCIIString", new Class[0]); + java.lang.reflect.Method toASCIIStringMethod + = uriClazz.getMethod("toASCIIString", new Class[0]); return (String) toASCIIStringMethod.invoke(uriObj, new Object[] {}); } catch (Exception e) { // Reflection problems? Should not happen, debug. Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/GlobPatternMapper.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/GlobPatternMapper.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/GlobPatternMapper.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/GlobPatternMapper.java Sun Nov 12 14:39:00 2006 @@ -33,6 +33,7 @@ */ public class GlobPatternMapper implements FileNameMapper { + // CheckStyle:VisibilityModifier OFF - bc /** * Part of "from" pattern before the *. */ @@ -62,6 +63,8 @@ * Part of "to" pattern after the *. */ protected String toPostfix = null; + + // CheckStyle:VisibilityModifier ON private boolean handleDirSep = false; private boolean caseSensitive = true; Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/IdentityStack.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/IdentityStack.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/IdentityStack.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/IdentityStack.java Sun Nov 12 14:39:00 2006 @@ -60,6 +60,7 @@ /** * Override methods that use <code>.equals()</code> comparisons on elements. * @param o the Object to search for. + * @return true if the stack contains the object. * @see java.util.Vector#contains(Object) */ public synchronized boolean contains(Object o) { @@ -70,6 +71,7 @@ * Override methods that use <code>.equals()</code> comparisons on elements. * @param o the Object to search for. * @param pos the position from which to search. + * @return the position of the object, -1 if not found. * @see java.util.Vector#indexOf(Object, int) */ public synchronized int indexOf(Object o, int pos) { @@ -85,6 +87,7 @@ * Override methods that use <code>.equals()</code> comparisons on elements. * @param o the Object to search for. * @param pos the position from which to search (backward). + * @return the position of the object, -1 if not found. * @see java.util.Vector#indexOf(Object, int) */ public synchronized int lastIndexOf(Object o, int pos) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/JavaEnvUtils.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/JavaEnvUtils.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/JavaEnvUtils.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/JavaEnvUtils.java Sun Nov 12 14:39:00 2006 @@ -342,7 +342,9 @@ switch(javaVersionNumber) { case 16: case 15: - tests.addElement("com.sun.org.apache.xerces.internal.jaxp.datatype.DatatypeFactoryImpl "); + tests.addElement( + "com.sun.org.apache.xerces.internal.jaxp.datatype.DatatypeFactoryImpl "); + // Fall tru case 14: tests.addElement("sun.audio.AudioPlayer"); if (javaVersionNumber == 14) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/LazyFileOutputStream.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/LazyFileOutputStream.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/LazyFileOutputStream.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/LazyFileOutputStream.java Sun Nov 12 14:39:00 2006 @@ -41,6 +41,7 @@ /** * Creates a stream that will eventually write to the file with * the given name and replace it. + * @param name the filename. */ public LazyFileOutputStream(String name) { this(name, false); @@ -50,6 +51,8 @@ * Creates a stream that will eventually write to the file with * the given name and optionally append to instead of replacing * it. + * @param name the filename. + * @param append if true append rather than replace. */ public LazyFileOutputStream(String name, boolean append) { this(new File(name), append); @@ -58,6 +61,7 @@ /** * Creates a stream that will eventually write to the file with * the given name and replace it. + * @param f the file to create. */ public LazyFileOutputStream(File f) { this(f, false); @@ -67,6 +71,8 @@ * Creates a stream that will eventually write to the file with * the given name and optionally append to instead of replacing * it. + * @param file the file to create. + * @param append if true append rather than replace. */ public LazyFileOutputStream(File file, boolean append) { this(file, append, false); @@ -76,6 +82,9 @@ * Creates a stream that will eventually write to the file with * the given name, optionally append to instead of replacing * it, and optionally always create a file (even if zero length). + * @param file the file to create. + * @param append if true append rather than replace. + * @param alwaysCreate if true create the file even if nothing to write. */ public LazyFileOutputStream(File file, boolean append, boolean alwaysCreate) { @@ -88,6 +97,7 @@ * Explicitly open the file for writing. * * <p>Returns silently if the file has already been opened.</p> + * @throws IOException if there is an error. */ public void open() throws IOException { ensureOpened(); Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java Sun Nov 12 14:39:00 2006 @@ -29,8 +29,11 @@ * @since Ant 1.6 */ public class LazyHashtable extends Hashtable { + // CheckStyle:VisibilityModifier OFF - bc protected boolean initAllDone = false; + // CheckStyle:VisibilityModifier OFF - bc + /** No arg constructor. */ public LazyHashtable() { super(); } Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/LeadPipeInputStream.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/LeadPipeInputStream.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/LeadPipeInputStream.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/LeadPipeInputStream.java Sun Nov 12 14:39:00 2006 @@ -67,6 +67,7 @@ * circular buffer of the specified size. * @param src the <code>PipedOutputStream</code> source. * @param size the size of the circular buffer. + * @throws IOException if there is an error. */ public LeadPipeInputStream(PipedOutputStream src, int size) throws IOException { super(src); Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/LineOrientedOutputStream.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/LineOrientedOutputStream.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/LineOrientedOutputStream.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/LineOrientedOutputStream.java Sun Nov 12 14:39:00 2006 @@ -48,6 +48,7 @@ * separator is detected. * * @param cc data to log (byte). + * @throws IOException if there is an error. */ public final void write(int cc) throws IOException { final byte c = (byte) cc; @@ -63,6 +64,7 @@ /** * Flush this log stream + * @throws IOException if there is an error. */ public final void flush() throws IOException { if (buffer.size() > 0) { @@ -73,6 +75,7 @@ /** * Converts the buffer to a string and sends it to * <code>processLine</code> + * @throws IOException if there is an error. */ protected void processBuffer() throws IOException { try { @@ -86,11 +89,13 @@ * Processes a line. * * @param line the line to log. + * @throws IOException if there is an error. */ protected abstract void processLine(String line) throws IOException; /** * Writes all remaining + * @throws IOException if there is an error. */ public final void close() throws IOException { if (buffer.size() > 0) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/MergingMapper.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/MergingMapper.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/MergingMapper.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/MergingMapper.java Sun Nov 12 14:39:00 2006 @@ -27,7 +27,9 @@ * */ public class MergingMapper implements FileNameMapper { + // CheckStyle:VisibilityModifier OFF - bc protected String[] mergedFile = null; + // CheckStyle:VisibilityModifier ON /** * Ignored. Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/ProxySetup.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/ProxySetup.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/ProxySetup.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/ProxySetup.java Sun Nov 12 14:39:00 2006 @@ -38,26 +38,41 @@ * @value */ public static final String USE_SYSTEM_PROXIES = "java.net.useSystemProxies"; + /** the http proxyhost property */ public static final String HTTP_PROXY_HOST = "http.proxyHost"; + /** the http proxyport property */ public static final String HTTP_PROXY_PORT = "http.proxyPort"; + /** the https proxyhost property */ public static final String HTTPS_PROXY_HOST = "https.proxyHost"; + /** the https proxyport property */ public static final String HTTPS_PROXY_PORT = "https.proxyPort"; + /** the ftp proxyhost property */ public static final String FTP_PROXY_HOST = "ftp.proxyHost"; + /** the ftp proxyport property */ public static final String FTP_PROXY_PORT = "ftp.proxyPort"; + /** the ftp proxyport property */ public static final String HTTP_NON_PROXY_HOSTS = "http.nonProxyHosts"; + /** the http hosts not to be proxied property */ public static final String HTTPS_NON_PROXY_HOSTS = "https.nonProxyHosts"; + /** the ftp hosts not to be proxied property */ public static final String FTP_NON_PROXY_HOSTS = "ftp.nonProxyHosts"; + /** the http proxy username property */ public static final String HTTP_PROXY_USERNAME = "http.proxyUser"; + /** the http proxy password property */ public static final String HTTP_PROXY_PASSWORD = "http.proxyPassword"; + /** the socks proxy host property */ public static final String SOCKS_PROXY_HOST = "socksProxyHost"; + /** the socks proxy port property */ public static final String SOCKS_PROXY_PORT = "socksProxyPort"; + /** the socks proxy username property */ public static final String SOCKS_PROXY_USERNAME = "java.net.socks.username"; + /** the socks proxy password property */ public static final String SOCKS_PROXY_PASSWORD = "java.net.socks.password"; /** * create a proxy setup class bound to this project - * @param owner + * @param owner the project that owns this setup. */ public ProxySetup(Project owner) { this.owner = owner; Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java Sun Nov 12 14:39:00 2006 @@ -29,9 +29,11 @@ * */ public class RegexpPatternMapper implements FileNameMapper { + // CheckStyle:VisibilityModifier OFF - bc protected RegexpMatcher reg = null; protected char[] to = null; protected StringBuffer result = new StringBuffer(); + // CheckStyle:VisibilityModifier ON /** * Constructor for RegexpPatternMapper. Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/RetryHandler.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/RetryHandler.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/RetryHandler.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/RetryHandler.java Sun Nov 12 14:39:00 2006 @@ -64,7 +64,8 @@ + this.retriesAllowed + "), giving up", Project.MSG_WARN); throw e; } else { - task.log("try #" + retries + ": IO error (" + desc + "), retrying", Project.MSG_WARN); + task.log("try #" + retries + ": IO error (" + desc + + "), retrying", Project.MSG_WARN); } } } Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/Retryable.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/Retryable.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/Retryable.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/Retryable.java Sun Nov 12 14:39:00 2006 @@ -27,7 +27,12 @@ * @see RetryHandler */ public interface Retryable { + /** The value to use to never give up. */ public static final int RETRY_FOREVER = -1; + /** + * Called to execute the code. + * @throws IOException if there is a problem. + */ void execute() throws IOException; } Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/SourceFileScanner.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/SourceFileScanner.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/SourceFileScanner.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/SourceFileScanner.java Sun Nov 12 14:39:00 2006 @@ -36,7 +36,9 @@ */ public class SourceFileScanner implements ResourceFactory { + // CheckStyle:VisibilityModifier OFF - bc protected Task task; + // CheckStyle:VisibilityModifier ON private static final FileUtils FILE_UTILS = FileUtils.getFileUtils(); private File destDir; // base directory of the fileset Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/StringTokenizer.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/StringTokenizer.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/StringTokenizer.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/StringTokenizer.java Sun Nov 12 14:39:00 2006 @@ -20,7 +20,6 @@ import java.io.IOException; import java.io.Reader; import org.apache.tools.ant.ProjectComponent; -import org.apache.tools.ant.util.Tokenizer; /** * Class to tokenize the input as areas separated Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/StringUtils.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/StringUtils.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/StringUtils.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/StringUtils.java Sun Nov 12 14:39:00 2006 @@ -100,8 +100,10 @@ } /** - * Checks that a string buffer ends up with a given string. It may sound trivial with the existing - * JDK API but the various implementation among JDKs can make those methods extremely resource intensive + * Checks that a string buffer ends up with a given string. It may sound + * trivial with the existing + * JDK API but the various implementation among JDKs can make those + * methods extremely resource intensive * and perform poorly due to massive memory allocation and copying. See * @param buffer the buffer to perform the check on * @param suffix the suffix Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/UUEncoder.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/UUEncoder.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/UUEncoder.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/UUEncoder.java Sun Nov 12 14:39:00 2006 @@ -54,6 +54,7 @@ * input stream. * @param is the input stream. * @param out the output stream. + * @throws IOException if there is an error. */ public void encode(InputStream is, OutputStream out) throws IOException { Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/WeakishReference.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/WeakishReference.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/WeakishReference.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/WeakishReference.java Sun Nov 12 14:39:00 2006 @@ -22,8 +22,10 @@ import java.lang.ref.WeakReference; /** - * These classes are part of some code to reduce memory leaks by only retaining weak references to things - * on Java1.2+, and yet still work (with leaky hard references) on Java1.1. Now that Ant is 1.2+ only, + * These classes are part of some code to reduce memory leaks by only + * retaining weak references to things + * on Java1.2+, and yet still work (with leaky hard references) on Java1.1. + * Now that Ant is 1.2+ only, * life is simpler and none of the classes are needed any more. * * They are only retained in case a third-party task uses them Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/XmlConstants.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/XmlConstants.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/XmlConstants.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/XmlConstants.java Sun Nov 12 14:39:00 2006 @@ -25,18 +25,26 @@ */ public class XmlConstants { + /** property for location of xml schema */ public static final String PROPERTY_SCHEMA_LOCATION = "http://apache.org/xml/properties/schema/external-schemaLocation"; + /** property for location of no-name schema */ public static final String PROPERTY_NO_NAMESPACE_SCHEMA_LOCATION = "http://apache.org/xml/properties/schema/external-noNamespaceSchemaLocation"; + /** property for full validation */ public static final String FEATURE_XSD_FULL_VALIDATION = "http://apache.org/xml/features/validation/schema-full-checking"; + /** property for xsd */ public static final String FEATURE_XSD = "http://apache.org/xml/features/validation/schema"; + /** property for validation */ public static final String FEATURE_VALIDATION = "http://xml.org/sax/features/validation"; + /** property for namespace support */ public static final String FEATURE_NAMESPACES = "http://xml.org/sax/features/namespaces"; + /** property for schema language */ public static final String FEATURE_JAXP12_SCHEMA_LANGUAGE = "http://java.sun.com/xml/jaxp/properties/schemaLanguage"; + /** property for schema source */ public static final String FEATURE_JAXP12_SCHEMA_SOURCE = "http://java.sun.com/xml/jaxp/properties/schemaSource"; public static final String URI_XSD = Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/java15/ProxyDiagnostics.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/java15/ProxyDiagnostics.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/java15/ProxyDiagnostics.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/java15/ProxyDiagnostics.java Sun Nov 12 14:39:00 2006 @@ -40,9 +40,9 @@ */ public class ProxyDiagnostics { - String destination; + private String destination; - URI destURI; + private URI destURI; /** [EMAIL PROTECTED] */ public static final String DEFAULT_DESTINATION = "http://ant.apache.org/"; @@ -69,6 +69,10 @@ this(DEFAULT_DESTINATION); } + /** + * Get the diagnostics for proxy information. + * @return the information. + */ public String toString() { ProxySelector selector = ProxySelector.getDefault(); List list = selector.select(destURI); Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/optional/ScriptRunner.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/optional/ScriptRunner.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/optional/ScriptRunner.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/optional/ScriptRunner.java Sun Nov 12 14:39:00 2006 @@ -24,7 +24,6 @@ import org.apache.bsf.BSFException; import org.apache.bsf.BSFManager; -import org.apache.tools.ant.AntClassLoader; import org.apache.tools.ant.BuildException; import org.apache.tools.ant.ProjectComponent; import org.apache.tools.ant.Project; @@ -187,6 +186,7 @@ /** * Set the class path to be used. + * @param classpath the path to use. */ public void setClasspath(Path classpath) { this.classpath = classpath; Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/optional/WeakishReference12.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/optional/WeakishReference12.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/optional/WeakishReference12.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/optional/WeakishReference12.java Sun Nov 12 14:39:00 2006 @@ -20,8 +20,6 @@ import org.apache.tools.ant.util.WeakishReference; -import java.lang.ref.WeakReference; - /** * This is a reference that really is is Weak, as it uses the * appropriate java.lang.ref class. Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroMatcher.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroMatcher.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroMatcher.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroMatcher.java Sun Nov 12 14:39:00 2006 @@ -32,8 +32,10 @@ public class JakartaOroMatcher implements RegexpMatcher { private String pattern; + // CheckStyle:VisibilityModifier OFF - bc protected final Perl5Compiler compiler = new Perl5Compiler(); protected final Perl5Matcher matcher = new Perl5Matcher(); + // CheckStyle:VisibilityModifier ON /** * Constructor for JakartaOroMatcher. Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java Sun Nov 12 14:39:00 2006 @@ -86,14 +86,16 @@ cause = orCause(cause, be, true); } - throw new BuildException("No supported regular expression matcher found" + - (cause != null ? ": " + cause : ""), cause); + throw new BuildException( + "No supported regular expression matcher found" + + (cause != null ? ": " + cause : ""), cause); } /** * Wrapper over RegexpMatcherFactory.createInstance that ensures that * we are dealing with a Regexp implementation. - * + * @return the instance. + * @throws BuildException if there is a problem. * @since 1.3 * * @see RegexpMatcherFactory#createInstance(String) Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java?view=diff&rev=474079&r1=474078&r2=474079 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java Sun Nov 12 14:39:00 2006 @@ -94,8 +94,9 @@ cause = orCause(cause, be, true); } - throw new BuildException("No supported regular expression matcher found" + - (cause != null ? ": " + cause : ""), cause); + throw new BuildException( + "No supported regular expression matcher found" + + (cause != null ? ": " + cause : ""), cause); } static Throwable orCause(Throwable deflt, BuildException be, boolean ignoreCnfe) { --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]