Author: peterreilly Date: Sun Nov 5 14:50:17 2006 New Revision: 471554 URL: http://svn.apache.org/viewvc?view=rev&rev=471554 Log: checkstyle
Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Javah.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/RenameExtensions.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ReplaceRegExp.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/StyleBook.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/TraXLiaison.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetCompile.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetResource.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/Ildasm.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/DeweyDecimal.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/JarLibResolveTask.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/Specification.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirror.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirrorImpl.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/DefaultNative2Ascii.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/Native2AsciiAdapterFactory.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/SunNative2Ascii.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPConfigurator.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/MimeMail.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDef.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDefBase.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/splash/SplashTask.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/AbstractSshMessage.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/SSHUserInfo.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessage.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessageBySftp.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessageBySftp.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/starteam/TreeBasedTask.java ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/unix/Symlink.java Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Javah.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Javah.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Javah.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Javah.java Sun Nov 5 14:50:17 2006 @@ -353,7 +353,7 @@ facade.setImplementation(JavahAdapterFactory.getDefault()); } else { facade.setImplementation(impl); - } + } } /** @@ -414,7 +414,7 @@ classpath = classpath.concatSystemClasspath("ignore"); } - JavahAdapter ad = + JavahAdapter ad = JavahAdapterFactory.getAdapter(facade.getImplementation(), this); if (!ad.compile(this)) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java Sun Nov 5 14:50:17 2006 @@ -131,7 +131,7 @@ facade.setImplementation(Native2AsciiAdapterFactory.getDefault()); } else { facade.setImplementation(impl); - } + } } /** @@ -233,7 +233,7 @@ * @param srcName name of the input file. * @param destName name of the input file. */ - private void convert(String srcName, String destName) + private void convert(String srcName, String destName) throws BuildException { File srcFile; // File to convert File destFile; // where to put the results @@ -261,7 +261,7 @@ } log("converting " + srcName, Project.MSG_VERBOSE); - Native2AsciiAdapter ad = + Native2AsciiAdapter ad = Native2AsciiAdapterFactory.getAdapter(facade.getImplementation(), this); if (!ad.convert(this, srcFile, destFile)) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/RenameExtensions.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/RenameExtensions.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/RenameExtensions.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/RenameExtensions.java Sun Nov 5 14:50:17 2006 @@ -39,7 +39,7 @@ * * @version 1.2 * - * @deprecated since 1.5.x. + * @deprecated since 1.5.x. * Use <move> instead */ public class RenameExtensions extends MatchingTask { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ReplaceRegExp.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ReplaceRegExp.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ReplaceRegExp.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ReplaceRegExp.java Sun Nov 5 14:50:17 2006 @@ -210,7 +210,7 @@ * each line, which is not easy to do when processing the file as a whole. * Defaults to <i>false</i>.</td> * @param byline the byline attribute as a string - * @deprecated since 1.6.x. + * @deprecated since 1.6.x. * Use setByLine(boolean). */ public void setByLine(String byline) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Rpm.java Sun Nov 5 14:50:17 2006 @@ -183,7 +183,7 @@ log("Building the RPM based on the " + specFile + " file"); int returncode = exe.execute(); if (Execute.isFailure(returncode)) { - String msg = "'" + toExecute.getExecutable() + String msg = "'" + toExecute.getExecutable() + "' failed with exit code " + returncode; if (failOnError) { throw new BuildException(msg); @@ -279,8 +279,8 @@ } /** - * If <code>true</code>, stop the build process when the rpmbuild command - * exits with an error status. + * If <code>true</code>, stop the build process when the rpmbuild command + * exits with an error status. * @param value <code>true</code> if it should halt, otherwise * <code>false</code>. The default is <code>false</code>. * @@ -291,7 +291,7 @@ } /** - * If true, output from the RPM build command will only be logged to DEBUG. + * If true, output from the RPM build command will only be logged to DEBUG. * @param value <code>false</code> if output should be logged, otherwise * <code>true</code>. The default is <code>false</code>. * Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/StyleBook.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/StyleBook.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/StyleBook.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/StyleBook.java Sun Nov 5 14:50:17 2006 @@ -30,7 +30,7 @@ * and options are available. Do not set any apart from the <tt>classpath</tt> * as they are not guaranteed to be there in future. * @todo stop extending from Java. - * @deprecated since 1.7. + * @deprecated since 1.7. * This task is considered unsupported by the Ant developers */ public class StyleBook extends Java { @@ -63,7 +63,7 @@ /** * Set the directory that contains the stylebook skin; * required. - * @param skinDirectory the location of the stylebook skin + * @param skinDirectory the location of the stylebook skin */ public void setSkinDirectory(final File skinDirectory) { m_skinDirectory = skinDirectory; Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/TraXLiaison.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/TraXLiaison.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/TraXLiaison.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/TraXLiaison.java Sun Nov 5 14:50:17 2006 @@ -337,7 +337,8 @@ * Sets the paramters for the transformer. */ private void setTransformationParameters() { - for (final Enumeration enumeration = params.keys(); enumeration.hasMoreElements(); ) { + for (final Enumeration enumeration = params.keys(); + enumeration.hasMoreElements();) { final String name = (String) enumeration.nextElement(); final String value = (String) params.get(name); transformer.setParameter(name, value); @@ -565,4 +566,4 @@ setOutputProperty(prop.getName(), prop.getValue()); } } -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetCompile.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetCompile.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetCompile.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetCompile.java Sun Nov 5 14:50:17 2006 @@ -914,8 +914,12 @@ while (files.hasMoreElements()) { File file = (File) files.nextElement(); if (isFileManagedBinary(file)) { - if (isWindows) command.addArgument('"'+REFERENCE_OPTION+file.toString()+'"'); - else command.addArgument(REFERENCE_OPTION+file.toString()); + if (isWindows) { + command.addArgument( + '"' + REFERENCE_OPTION + file.toString() + '"'); + } else { + command.addArgument(REFERENCE_OPTION + file.toString()); + } } else { log("ignoring " + file + " as it is not a managed executable", Project.MSG_VERBOSE); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetResource.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetResource.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetResource.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/DotnetResource.java Sun Nov 5 14:50:17 2006 @@ -68,7 +68,7 @@ /** * embed the resource in the assembly (default, true) or just link to it. - * + * * @param embed */ public void setEmbed(boolean embed) { @@ -81,7 +81,7 @@ /** * name the resource - * + * * @param file */ public void setFile(File file) { @@ -94,7 +94,7 @@ /** * VB and J# only: is a resource public or not? - * + * * @param aPublic */ public void setPublic(Boolean aPublic) { @@ -107,7 +107,7 @@ /** * should the resource have a name? - * + * * @param name */ public void setName(String name) { @@ -116,7 +116,7 @@ /** * Filesets root namespace. The value always ends with '.' . - * + * * @return String namespace name */ public String getNamespace() { @@ -125,7 +125,7 @@ /** * Sets filesets root namespace. - * + * * @param namespace * String root namespace */ @@ -206,7 +206,7 @@ /** * Adds a resource file set. - * + * * @param fileset * FileSet */ @@ -216,10 +216,10 @@ /** * Checks that <resource> node has embedded <filesets> - * + * * @return boolean */ public boolean hasFilesets() { return fileSets.size() > 0; } -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/Ildasm.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/Ildasm.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/Ildasm.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/dotnet/Ildasm.java Sun Nov 5 14:50:17 2006 @@ -60,7 +60,7 @@ public class Ildasm extends Task { private static final FileUtils FILE_UTILS = FileUtils.getFileUtils(); - + /** * source file (mandatory) */ Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/DeweyDecimal.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/DeweyDecimal.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/DeweyDecimal.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/DeweyDecimal.java Sun Nov 5 14:50:17 2006 @@ -25,11 +25,11 @@ * "1.2.3.4.5.6.7". This allows an extensible number to be used to * represent major, minor, micro, etc versions. The version number * must begin with a number. - * + * * Original Implementation moved to org.apache.tools.ant.util.DeweyDecimal */ public final class DeweyDecimal extends org.apache.tools.ant.util.DeweyDecimal { - + /** * Construct a DeweyDecimal from an array of integer components. * Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/JarLibResolveTask.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/JarLibResolveTask.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/JarLibResolveTask.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/JarLibResolveTask.java Sun Nov 5 14:50:17 2006 @@ -156,7 +156,7 @@ final String message = "Property Already set to: " + candidate; if (failOnError) { throw new BuildException(message); - } + } getProject().log(message, Project.MSG_ERR); return; } @@ -282,4 +282,4 @@ throw new BuildException(message); } } -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/Specification.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/Specification.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/Specification.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/extension/Specification.java Sun Nov 5 14:50:17 2006 @@ -39,9 +39,9 @@ * */ public final class Specification { - + private static final String MISSING = "Missing "; - + /** * Manifest Attribute Name object for SPECIFICATION_TITLE. */ @@ -325,7 +325,7 @@ public String[] getSections() { if (null == sections) { return null; - } + } final String[] newSections = new String[ sections.length ]; System.arraycopy(sections, 0, newSections, 0, sections.length); return newSections; @@ -526,10 +526,10 @@ return specification; } sectionsToAdd.addAll(Arrays.asList(specification.getSections())); - + final String[] sections = (String[]) sectionsToAdd.toArray(new String[sectionsToAdd.size()]); - + return new Specification(specification.getSpecificationTitle(), specification.getSpecificationVersion().toString(), specification.getSpecificationVendor(), Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/javah/SunJavah.java Sun Nov 5 14:50:17 2006 @@ -54,8 +54,8 @@ c = Class.forName("com.sun.tools.javah.Main"); } } catch (ClassNotFoundException ex) { - throw new BuildException("Can't load javah", ex, - javah.getLocation()); + throw new BuildException( + "Can't load javah", ex, javah.getLocation()); } cmd.setExecutable(c.getName()); ej.setJavaCommand(cmd); @@ -94,8 +94,8 @@ cmd.createArgument().setValue("-force"); } if (javah.getStubs() && !javah.getOld()) { - throw new BuildException("stubs only available in old mode.", - javah.getLocation()); + throw new BuildException( + "stubs only available in old mode.", javah.getLocation()); } if (javah.getStubs()) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java Sun Nov 5 14:50:17 2006 @@ -948,8 +948,9 @@ + String.valueOf(showOutput)); cmd.createArgument().setValue(Constants.OUTPUT_TO_FORMATTERS + String.valueOf(outputToFormatters)); - - cmd.createArgument().setValue(Constants.LOGTESTLISTENEREVENTS+"true"); // #31885 + + cmd.createArgument().setValue( + Constants.LOGTESTLISTENEREVENTS + "true"); // #31885 StringBuffer formatterArg = new StringBuffer(STRING_BUFFER_SIZE); final FormatterElement[] feArray = mergeFormatters(test); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirror.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirror.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirror.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirror.java Sun Nov 5 14:50:17 2006 @@ -34,16 +34,15 @@ * - JUnitTestRunner * Cf. JUnitTask.SplitLoader#isSplit(String) * Public only to permit access from classes in this package; do not use directly. - * + * * @since 1.7 * @see "bug #38799" */ public interface JUnitTaskMirror { - + void addVmExit(JUnitTest test, JUnitResultFormatterMirror formatter, OutputStream out, String message, String testCase); - - + JUnitTestRunnerMirror newJUnitTestRunner(JUnitTest test, boolean haltOnError, boolean filterTrace, boolean haltOnFailure, boolean showOutput, boolean logTestListenerEvents, AntClassLoader classLoader); @@ -77,7 +76,7 @@ * No problems with this test. */ int SUCCESS = 0; - + /** * Some tests failed. */ @@ -89,23 +88,22 @@ int ERRORS = 2; void setPermissions(Permissions perm); - + void run(); void addFormatter(JUnitResultFormatterMirror formatter); - + int getRetCode(); - + void handleErrorFlush(String output); - + void handleErrorOutput(String output); - + void handleOutput(String output); - + int handleInput(byte[] buffer, int offset, int length) throws IOException; - + void handleFlush(String output); } - } Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirrorImpl.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirrorImpl.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirrorImpl.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTaskMirrorImpl.java Sun Nov 5 14:50:17 2006 @@ -93,7 +93,7 @@ } public String toString() { - return test.getName()+":"+testCase; + return test.getName() + ":" + testCase; } } } Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java Sun Nov 5 14:50:17 2006 @@ -188,7 +188,7 @@ public JUnitTestRunner(JUnitTest test, boolean haltOnError, boolean filtertrace, boolean haltOnFailure, boolean showOutput, boolean logTestListenerEvents) { - this(test, haltOnError, filtertrace, haltOnFailure, showOutput, + this(test, haltOnError, filtertrace, haltOnFailure, showOutput, logTestListenerEvents, null); } @@ -207,7 +207,7 @@ public JUnitTestRunner(JUnitTest test, boolean haltOnError, boolean filtertrace, boolean haltOnFailure, boolean showOutput, ClassLoader loader) { - this(test, haltOnError, filtertrace, haltOnFailure, showOutput, + this(test, haltOnError, filtertrace, haltOnFailure, showOutput, false, loader); } @@ -253,12 +253,14 @@ System.setOut( new PrintStream( new OutputStream() { - public void write(int b) {} + public void write(int b) { + } })); System.setErr( new PrintStream( new OutputStream() { - public void write(int b) {} + public void write(int b) { + } })); } } else { @@ -369,7 +371,7 @@ long start = System.currentTimeMillis(); fireStartTestSuite(); - startTestSuiteSuccess = true; + startTestSuiteSuccess = true; if (exception != null) { // had an exception constructing suite for (int i = 0; i < formatters.size(); i++) { ((TestListener) formatters.elementAt(i)) @@ -900,7 +902,7 @@ // We would prefer to show "failure" for things that logically are. try { String msg = t.getMessage(); - AssertionFailedError failure = msg != null + AssertionFailedError failure = msg != null ? new AssertionFailedError(msg) : new AssertionFailedError(); // To compile on pre-JDK 4 (even though this should always succeed): Method initCause = Throwable.class.getMethod("initCause", new Class[] {Throwable.class}); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java Sun Nov 5 14:50:17 2006 @@ -245,9 +245,10 @@ File file = files[i]; try { log("Parsing file: '" + file + "'", Project.MSG_VERBOSE); - if(file.length()>0) { + if (file.length() > 0) { Document testsuiteDoc - = builder.parse(FileUtils.getFileUtils().toURI(files[i].getAbsolutePath())); + = builder.parse( + FileUtils.getFileUtils().toURI(files[i].getAbsolutePath())); Element elem = testsuiteDoc.getDocumentElement(); // make sure that this is REALLY a testsuite. if (TESTSUITE.equals(elem.getNodeName())) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/DefaultNative2Ascii.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/DefaultNative2Ascii.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/DefaultNative2Ascii.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/DefaultNative2Ascii.java Sun Nov 5 14:50:17 2006 @@ -47,7 +47,7 @@ addFiles(cmd, args, srcFile, destFile); return run(cmd, args); } - + /** * Sets up the initial command line. * @@ -93,4 +93,4 @@ */ protected abstract boolean run(Commandline cmd, ProjectComponent log) throws BuildException; -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/Native2AsciiAdapterFactory.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/Native2AsciiAdapterFactory.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/Native2AsciiAdapterFactory.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/Native2AsciiAdapterFactory.java Sun Nov 5 14:50:17 2006 @@ -79,10 +79,9 @@ * isn't an instance of Native2AsciiAdapter. */ private static Native2AsciiAdapter resolveClassName(String className) - throws BuildException - { + throws BuildException { return (Native2AsciiAdapter) ClasspathUtils.newInstance(className, - Native2AsciiAdapterFactory.class.getClassLoader(), + Native2AsciiAdapterFactory.class.getClassLoader(), Native2AsciiAdapter.class); } -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/SunNative2Ascii.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/SunNative2Ascii.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/SunNative2Ascii.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/native2ascii/SunNative2Ascii.java Sun Nov 5 14:50:17 2006 @@ -54,7 +54,7 @@ + "sun.tools.native2ascii.Main"); } Object o = n2aMain.newInstance(); - return ((Boolean) convert.invoke(o, + return ((Boolean) convert.invoke(o, new Object[] {cmd.getArguments()}) ).booleanValue(); } catch (BuildException ex) { @@ -65,4 +65,4 @@ throw new BuildException("Error starting Sun's native2ascii: ", ex); } } -} \ No newline at end of file +} Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java Sun Nov 5 14:50:17 2006 @@ -901,7 +901,7 @@ } public String toString() { - return "AntFtpFile: "+curpwd+"%"+ftpFile; + return "AntFtpFile: " + curpwd + "%" + ftpFile; } } /** @@ -1058,10 +1058,10 @@ public void setPassword(String password) { this.password = password; } - + /** * Sets the login account to use on the specified server. - * + * * @param pAccount the account name on remote system * @since Ant 1.7 */ @@ -1215,7 +1215,7 @@ * Sets the FTP action to be taken. Currently accepts "put", "get", "del", * "mkdir", "chmod", "list", and "site". * - * @deprecated since 1.5.x. + * @deprecated since 1.5.x. * setAction(String) is deprecated and is replaced with * setAction(FTP.Action) to make Ant's Introspection mechanism do the * work and also to encapsulate operations on the type in its own @@ -2307,7 +2307,7 @@ public void execute() throws IOException { doSiteCommand(lftp, FTP.this.initialSiteCommand); } - }, "initial site command: "+ this.initialSiteCommand); + }, "initial site command: " + this.initialSiteCommand); } @@ -2362,7 +2362,7 @@ } } catch (IOException ex) { - throw new BuildException("error during FTP transfer: " + ex,ex); + throw new BuildException("error during FTP transfer: " + ex, ex); } finally { if (ftp != null && ftp.isConnected()) { try { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPConfigurator.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPConfigurator.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPConfigurator.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPConfigurator.java Sun Nov 5 14:50:17 2006 @@ -22,9 +22,9 @@ /** * The sole purpose of this class is (note that it is package-private - * is to serve as a separate, static compilation unit for importing - * FTPClientConfig, to enable users who wish to use the FTP task - * without using its new features to avoid the need to + * is to serve as a separate, static compilation unit for importing + * FTPClientConfig, to enable users who wish to use the FTP task + * without using its new features to avoid the need to * upgrade to jakarta-commons-net 1.4.0, where FTPClientConfig was * introduced. */ @@ -40,53 +40,52 @@ static FTPClient configure(FTPClient client, FTP task) { task.log("custom configuration", Project.MSG_VERBOSE); FTPClientConfig config; - String systemTypeKey = task.getSystemTypeKey(); + String systemTypeKey = task.getSystemTypeKey(); if (systemTypeKey != null && !"".equals(systemTypeKey)) { config = new FTPClientConfig(systemTypeKey); - task.log("custom config: system key = " + task.log("custom config: system key = " + systemTypeKey, Project.MSG_VERBOSE); } else { config = new FTPClientConfig(); - task.log("custom config: system key = default (UNIX)", + task.log("custom config: system key = default (UNIX)", Project.MSG_VERBOSE); } - + String defaultDateFormatConfig = task.getDefaultDateFormatConfig(); if (defaultDateFormatConfig != null) { config.setDefaultDateFormatStr(defaultDateFormatConfig); - task.log("custom config: default date format = " + task.log("custom config: default date format = " + defaultDateFormatConfig, Project.MSG_VERBOSE); } - + String recentDateFormatConfig = task.getRecentDateFormatConfig(); if (recentDateFormatConfig != null) { config.setRecentDateFormatStr(recentDateFormatConfig); - task.log("custom config: recent date format = " + task.log("custom config: recent date format = " + recentDateFormatConfig, Project.MSG_VERBOSE); } - + String serverLanguageCodeConfig = task.getServerLanguageCodeConfig(); if (serverLanguageCodeConfig != null) { config.setServerLanguageCode(serverLanguageCodeConfig); - task.log("custom config: server language code = " + task.log("custom config: server language code = " + serverLanguageCodeConfig, Project.MSG_VERBOSE); } - + String serverTimeZoneConfig = task.getServerTimeZoneConfig(); if (serverTimeZoneConfig != null) { config.setServerTimeZoneId(serverTimeZoneConfig); - task.log("custom config: server time zone ID = " + task.log("custom config: server time zone ID = " + serverTimeZoneConfig, Project.MSG_VERBOSE); } - + String shortMonthNamesConfig = task.getShortMonthNamesConfig(); if (shortMonthNamesConfig != null) { config.setShortMonthNames(shortMonthNamesConfig); - task.log("custom config: short month names = " + task.log("custom config: short month names = " + shortMonthNamesConfig, Project.MSG_VERBOSE); } client.configure(config); return client; - } } Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/MimeMail.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/MimeMail.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/MimeMail.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/MimeMail.java Sun Nov 5 14:50:17 2006 @@ -23,7 +23,7 @@ /** * A task to send SMTP email; Use <tt>mail</tt> instead * - * @deprecated since 1.6.x. + * @deprecated since 1.6.x. * Use [EMAIL PROTECTED] EmailTask} instead. * * @since Ant1.4 Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDef.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDef.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDef.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDef.java Sun Nov 5 14:50:17 2006 @@ -296,11 +296,11 @@ * * @param attributes collection of attributes * @param elements a list of nested element values. - * @deprecated since 1.7. + * @deprecated since 1.7. * Use executeScript(attribute, elements, instance) instead. */ public void executeScript(Map attributes, Map elements) { - executeScript(attributes, elements,null); + executeScript(attributes, elements, null); } /** @@ -316,7 +316,7 @@ runner.addBean("attributes", attributes); runner.addBean("elements", elements); runner.addBean("project", getProject()); - if(instance!=null) { + if (instance != null) { runner.addBean("self", instance); } runner.executeScript("scriptdef_" + name); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDefBase.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDefBase.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDefBase.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/script/ScriptDefBase.java Sun Nov 5 14:50:17 2006 @@ -39,7 +39,7 @@ /** Attributes */ private Map attributes = new HashMap(); - + private String text; /** @@ -105,7 +105,7 @@ * @since ant1.7 */ public void addText(String text) { - this.text=getProject().replaceProperties(text); + this.text = getProject().replaceProperties(text); } /** Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/splash/SplashTask.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/splash/SplashTask.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/splash/SplashTask.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/splash/SplashTask.java Sun Nov 5 14:50:17 2006 @@ -62,7 +62,7 @@ * flag to enable proxy settings; optional, deprecated : consider * using <setproxy> instead * @param useProxy if ture, enable proxy settings - * @deprecated since 1.5.x. + * @deprecated since 1.5.x. * Use org.apache.tools.ant.taskdefs.optional.SetProxy */ public void setUseproxy(boolean useProxy) { Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/AbstractSshMessage.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/AbstractSshMessage.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/AbstractSshMessage.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/AbstractSshMessage.java Sun Nov 5 14:50:17 2006 @@ -251,8 +251,8 @@ public boolean count(long len) { totalLength += len; - percentTransmitted = trackProgress(initFileSize, - totalLength, + percentTransmitted = trackProgress(initFileSize, + totalLength, percentTransmitted); return true; } @@ -261,7 +261,7 @@ } public long getTotalLength() { - return totalLength; + return totalLength; } } } Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/SSHUserInfo.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/SSHUserInfo.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/SSHUserInfo.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/SSHUserInfo.java Sun Nov 5 14:50:17 2006 @@ -189,7 +189,7 @@ public void showMessage(String message) { //log(message, Project.MSG_DEBUG); } - + public String[] promptKeyboardInteractive(String destination, String name, String instruction, @@ -198,8 +198,8 @@ if (prompt.length != 1 || echo[0] != false || this.password == null) { return null; } - String[] response=new String[1]; - response[0]=this.password; + String[] response = new String[1]; + response[0] = this.password; return response; } Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessage.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessage.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessage.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessage.java Sun Nov 5 14:50:17 2006 @@ -216,7 +216,7 @@ try { while (true) { length = in.read(buf, 0, - (BUFFER_SIZE < filesize) ? BUFFER_SIZE + (BUFFER_SIZE < filesize) ? BUFFER_SIZE : (int) filesize); if (length < 0) { throw new EOFException("Unexpected end of stream."); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessageBySftp.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessageBySftp.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessageBySftp.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessageBySftp.java Sun Nov 5 14:50:17 2006 @@ -127,8 +127,8 @@ if (name.equals(".") || name.equals("..")) { continue; } - getDir(channel, - channel.pwd() + "/" + name + "/", + getDir(channel, + channel.pwd() + "/" + name + "/", new File(localFile, le.getFilename())); } else{ getFile(channel, le, localFile); @@ -137,7 +137,7 @@ channel.cd(".."); } - private void getFile(ChannelSftp channel, + private void getFile(ChannelSftp channel, ChannelSftp.LsEntry le, File localFile) throws IOException, SftpException { String remoteFile = le.getFilename(); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessageBySftp.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessageBySftp.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessageBySftp.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessageBySftp.java Sun Nov 5 14:50:17 2006 @@ -185,7 +185,7 @@ try{ channel.stat(dir); } - catch(SftpException e){ + catch (SftpException e) { // dir does not exist. if (e.id==ChannelSftp.SSH_FX_NO_SUCH_FILE) { channel.mkdir(dir); Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/starteam/TreeBasedTask.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/starteam/TreeBasedTask.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/starteam/TreeBasedTask.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/starteam/TreeBasedTask.java Sun Nov 5 14:50:17 2006 @@ -462,7 +462,7 @@ protected boolean isUsingViewLabel() { return null != this.labelInUse && this.labelInUse.isViewLabel(); } - + /** * returns true if a label has been specified and it is a revision label. * Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/unix/Symlink.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/unix/Symlink.java?view=diff&rev=471554&r1=471553&r2=471554 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/unix/Symlink.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/unix/Symlink.java Sun Nov 5 14:50:17 2006 @@ -263,7 +263,7 @@ File dir = (File) dirs.next(); Vector linksInDir = (Vector) byDir.get(dir); Properties linksToStore = new Properties(); - + // fill up a Properties object with link and resource names: for (Iterator dlnk = linksInDir.iterator(); dlnk.hasNext();) { File lnk = (File) dlnk.next(); --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]