Author: mbenson Date: Thu Jun 21 21:12:02 2007 New Revision: 549703 URL: http://svn.apache.org/viewvc?view=rev&rev=549703 Log: fmting
Modified: ant/core/trunk/src/main/org/apache/tools/ant/PropertyHelper.java Modified: ant/core/trunk/src/main/org/apache/tools/ant/PropertyHelper.java URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/PropertyHelper.java?view=diff&rev=549703&r1=549702&r2=549703 ============================================================================== --- ant/core/trunk/src/main/org/apache/tools/ant/PropertyHelper.java (original) +++ ant/core/trunk/src/main/org/apache/tools/ant/PropertyHelper.java Thu Jun 21 21:12:02 2007 @@ -15,14 +15,12 @@ * limitations under the License. * */ - package org.apache.tools.ant; import java.util.Hashtable; import java.util.Vector; import java.util.Enumeration; - /* ISSUES: - ns param. It could be used to provide "namespaces" for properties, which may be more flexible. @@ -131,7 +129,7 @@ */ public static synchronized PropertyHelper getPropertyHelper(Project project) { PropertyHelper helper - = (PropertyHelper) project.getReference(MagicNames.REFID_PROPERTY_HELPER); + = (PropertyHelper) project.getReference(MagicNames.REFID_PROPERTY_HELPER); if (helper != null) { return helper; } @@ -170,8 +168,7 @@ boolean inherited, boolean user, boolean isNew) { if (getNext() != null) { - boolean subst = getNext().setPropertyHook(ns, name, value, - inherited, user, isNew); + boolean subst = getNext().setPropertyHook(ns, name, value, inherited, user, isNew); // If next has handled the property if (subst) { return true; @@ -230,8 +227,7 @@ * <code>}</code> */ public void parsePropertyString(String value, Vector fragments, - Vector propertyRefs) - throws BuildException { + Vector propertyRefs) throws BuildException { parsePropertyStringDefault(value, fragments, propertyRefs); } @@ -253,8 +249,7 @@ * @return the original string with the properties replaced, or * <code>null</code> if the original string is <code>null</code>. */ - public String replaceProperties(String ns, String value, Hashtable keys) - throws BuildException { + public String replaceProperties(String ns, String value, Hashtable keys) throws BuildException { if (value == null || value.indexOf('$') == -1) { return value; } @@ -285,8 +280,7 @@ + "\" has not been set", Project.MSG_VERBOSE); } fragment = (replacement != null) - ? replacement.toString() - : "${" + propertyName + "}"; + ? replacement.toString() : "${" + propertyName + "}"; } sb.append(fragment); } @@ -297,7 +291,6 @@ // Methods used to support the default behavior and provide backward // compatibility. Some will be deprecated, you should avoid calling them. - /** Default implementation of setProperty. Will be called from Project. * This is the original 1.5 implementation, with calls to the hook * added. @@ -313,7 +306,7 @@ if (null != userProperties.get(name)) { if (verbose) { project.log("Override ignored for user property \"" + name - + "\"", Project.MSG_VERBOSE); + + "\"", Project.MSG_VERBOSE); } return false; } @@ -325,12 +318,12 @@ if (null != properties.get(name) && verbose) { project.log("Overriding previous definition of property \"" + name - + "\"", Project.MSG_VERBOSE); + + "\"", Project.MSG_VERBOSE); } if (verbose) { project.log("Setting project property: " + name + " -> " - + value, Project.MSG_DEBUG); + + value, Project.MSG_DEBUG); } if (name != null && value != null) { properties.put(name, value); @@ -354,17 +347,15 @@ Object value) { if (null != properties.get(name)) { project.log("Override ignored for property \"" + name - + "\"", Project.MSG_VERBOSE); + + "\"", Project.MSG_VERBOSE); return; } - boolean done = setPropertyHook(ns, name, value, false, false, true); if (done) { return; } - project.log("Setting project property: " + name + " -> " - + value, Project.MSG_DEBUG); + + value, Project.MSG_DEBUG); if (name != null && value != null) { properties.put(name, value); } @@ -382,7 +373,7 @@ public synchronized void setUserProperty(String ns, String name, Object value) { project.log("Setting ro project property: " + name + " -> " - + value, Project.MSG_DEBUG); + + value, Project.MSG_DEBUG); userProperties.put(name, value); boolean done = setPropertyHook(ns, name, value, false, true, false); @@ -409,7 +400,7 @@ inheritedProperties.put(name, value); project.log("Setting ro project property: " + name + " -> " - + value, Project.MSG_DEBUG); + + value, Project.MSG_DEBUG); userProperties.put(name, value); boolean done = setPropertyHook(ns, name, value, true, false, false); @@ -436,12 +427,10 @@ if (name == null) { return null; } - Object o = getPropertyHook(ns, name, false); if (o != null) { return o; } - return properties.get(name); } /** @@ -465,7 +454,6 @@ return userProperties.get(name); } - // -------------------- Access to property tables -------------------- // This is used to support ant call and similar tasks. It should be // deprecated, it is possible to use a better (more efficient) @@ -473,8 +461,7 @@ /** * Returns a copy of the properties table. - * @return a hashtable containing all properties - * (including user properties). + * @return a hashtable containing all properties (including user properties). */ public Hashtable getProperties() { //avoid concurrent modification: @@ -497,8 +484,7 @@ } /** - * special back door for subclasses, internal access to - * the hashtables + * special back door for subclasses, internal access to the hashtables * @return the live hashtable of all properties */ protected Hashtable getInternalProperties() { @@ -506,8 +492,7 @@ } /** - * special back door for subclasses, internal access to - * the hashtables + * special back door for subclasses, internal access to the hashtables * * @return the live hashtable of user properties */ @@ -516,8 +501,7 @@ } /** - * special back door for subclasses, internal access to - * the hashtables + * special back door for subclasses, internal access to the hashtables * * @return the live hashtable inherited properties */ @@ -525,7 +509,6 @@ return inheritedProperties; } - /** * Copies all user properties that have not been set on the * command line or a GUI tool from this instance to the Project @@ -588,9 +571,8 @@ /** Default parsing method. It is here only to support backward compatibility * for the static ProjectHelper.parsePropertyString(). */ - static void parsePropertyStringDefault(String value, Vector fragments, - Vector propertyRefs) - throws BuildException { + static void parsePropertyStringDefault(String value, Vector fragments, Vector propertyRefs) + throws BuildException { int prev = 0; int pos; //search for the next instance of $ from the 'prev' position @@ -624,13 +606,11 @@ fragments.addElement(value.substring(pos, pos + 2)); prev = pos + 2; } - } else { //property found, extract its name or bail on a typo int endName = value.indexOf('}', pos); if (endName < 0) { - throw new BuildException("Syntax error in property: " - + value); + throw new BuildException("Syntax error in property: " + value); } String propertyName = value.substring(pos + 2, endName); fragments.addElement(null); --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]