Author: mbenson Date: Tue Aug 29 07:43:54 2006 New Revision: 438081 URL: http://svn.apache.org/viewvc?rev=438081&view=rev Log: moved space append so that final deletion is unnecessary. Minor javadoc and formatting changes.
Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ManifestClassPath.java Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ManifestClassPath.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ManifestClassPath.java?rev=438081&r1=438080&r2=438081&view=diff ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ManifestClassPath.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ManifestClassPath.java Tue Aug 29 07:43:54 2006 @@ -1,5 +1,5 @@ /* - * Copyright 2005-2006 The Apache Software Foundation + * Copyright 2005-2006 The Apache Software Foundation * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -32,8 +32,7 @@ * * @ant.task category="property" */ -public class ManifestClassPath - extends Task { +public class ManifestClassPath extends Task { /** The property name to hold the classpath value. */ private String _name; @@ -48,7 +47,7 @@ private Path _path; /** - * Sets a property, which must not already exists, with a space + * Sets a property, which must not already exist, with a space * separated list of files and directories relative to the jar * file's parent directory. */ @@ -87,6 +86,9 @@ StringBuffer buffer = new StringBuffer(); StringBuffer element = new StringBuffer(); for (int i = 0; i < elements.length; ++i) { + if (i > 0) { + buffer.append(' '); + } // Normalize the current file File pathEntry = new File(elements[i]); pathEntry = fileUtils.normalize(pathEntry.getAbsolutePath()); @@ -134,12 +136,6 @@ throw new BuildException(exc); } buffer.append(relPath); - buffer.append(' '); - } - - // Get rid of trailing space, if any - if (buffer.length() > 0) { - buffer.setLength(buffer.length() - 1); } // Finally assign the property with the manifest classpath @@ -157,7 +153,7 @@ /** * The JAR file to contain the classpath attribute in its manifest. - * + * * @param jarfile the JAR file. Need not exist yet, but its parent * directory must exist on the other hand. */ --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]