[cp-patches] FYI: Fix to earlier patch

[Andrew John Hughes]

This fixes a mistake in the previous patch in one of the
constructors where all actions were actually performed
on defaultValue while the parameter was defValue.
Thanks to Raif for pointing this one out.

Changelog:

2006-08-01  Andrew John Hughes  <[EMAIL PROTECTED]>

        * javax/management/openmbean/OpenMBeanParameterInfoSupport.java:
        Call parameter 'defaultValue' not 'defValue'.

-- 
Andrew :-)

Escape the Java Trap with GNU Classpath!
http://www.gnu.org/philosophy/java-trap.html
public class gcj extends Freedom implements Java { ... }

Index: javax/management/openmbean/OpenMBeanParameterInfoSupport.java
===================================================================
RCS file: 
/cvsroot/classpath/classpath/javax/management/openmbean/OpenMBeanParameterInfoSupport.java,v
retrieving revision 1.1
diff -u -3 -p -u -r1.1 OpenMBeanParameterInfoSupport.java
--- javax/management/openmbean/OpenMBeanParameterInfoSupport.java       1 Aug 
2006 22:13:40 -0000       1.1
+++ javax/management/openmbean/OpenMBeanParameterInfoSupport.java       1 Aug 
2006 22:39:52 -0000
@@ -138,21 +138,21 @@ public class OpenMBeanParameterInfoSuppo
    * @param name the name of the parameter.
    * @param desc a description of the parameter.
    * @param type the open type of the parameter.
-   * @param defValue the default value of the parameter.
+   * @param defaultValue the default value of the parameter.
    * @throws IllegalArgumentException if the name, description or
    *                                  open type are <code>null</code>
    *                                  or the name or description are
    *                                  the empty string.
-   * @throws OpenDataException if <code>defValue<code> is non-null
+   * @throws OpenDataException if <code>defaultValue<code> is non-null
    *                           and is either not a value of the given
    *                           open type or the open type is an instance
    *                           of [EMAIL PROTECTED] ArrayType} or [EMAIL 
PROTECTED] TabularType}.
    */
   public OpenMBeanParameterInfoSupport(String name, String desc, OpenType type,
-                                      Object defValue)
+                                      Object defaultValue)
     throws OpenDataException
   {
-    this(name, desc, type, defValue, null);
+    this(name, desc, type, defaultValue, null);
   }
 
   /**
@@ -252,7 +252,7 @@ public class OpenMBeanParameterInfoSuppo
    * @param name the name of the parameter.
    * @param desc a description of the parameter.
    * @param type the open type of the parameter.
-   * @param defValue the default value of the parameter, or <code>null</code>.
+   * @param defaultValue the default value of the parameter, or 
<code>null</code>.
    * @param legalValues the legal values of the parameter.  May be
    *                    <code>null</code> or an empty array.
    * @throws IllegalArgumentException if the name, description or
@@ -262,7 +262,7 @@ public class OpenMBeanParameterInfoSuppo
    * @throws OpenDataException if any condition in the list above is broken.
    */
   public OpenMBeanParameterInfoSupport(String name, String desc, OpenType type,
-                                      Object defValue, Object[] legalValues)
+                                      Object defaultValue, Object[] 
legalValues)
     throws OpenDataException
   {
     super(name, type == null ? null : type.getClassName(), desc);

signature.asc
Description: Digital signature