[cp-patches] FYI: BeanContextSupport - constructor fixes

[David Gilbert]

This patch (committed) fixes a few failing Mauve tests:

2006-11-09  David Gilbert  <[EMAIL PROTECTED]>

   * java/beans/beancontext/BeanContextSupport.java
   (BeanContextSupport): Use correct dtime default,
   (BeanContextSupport(BeanContext)): Likewise,
   (BeanContextSupport(BeanContext, Locale)): Likewise, plus renamed
   locale argument,
   (BeanContextSupport(BeanContext, Locale, boolean)): Likewise,
   (BeanContextSupport(BeanContext, Locale, boolean, boolean)): Likewise.

Regards,

Dave

Index: java/beans/beancontext/BeanContextSupport.java
===================================================================
RCS file: 
/sources/classpath/classpath/java/beans/beancontext/BeanContextSupport.java,v
retrieving revision 1.14
diff -u -r1.14 BeanContextSupport.java
--- java/beans/beancontext/BeanContextSupport.java      9 Nov 2006 17:01:49 
-0000       1.14
+++ java/beans/beancontext/BeanContextSupport.java      9 Nov 2006 17:31:41 
-0000
@@ -144,42 +144,61 @@
    */
   public BeanContextSupport ()
   {
-    this (null, null, true, true);
+    this (null, null, false, true);
   }
 
   /**
    * Construct a BeanContextSupport instance.
+   * 
+   * @param peer  the bean context peer (<code>null</code> permitted).
    */
-  public BeanContextSupport (BeanContext peer)
+  public BeanContextSupport(BeanContext peer)
   {
-    this (peer, null, true, true);
+    this (peer, null, false, true);
   }
 
   /**
    * Construct a BeanContextSupport instance.
+   * 
+   * @param peer  the bean context peer (<code>null</code> permitted).
+   * @param locale  the locale (<code>null</code> permitted, equivalent to 
+   *     the default locale).
    */
-  public BeanContextSupport (BeanContext peer, Locale lcle)
+  public BeanContextSupport (BeanContext peer, Locale locale)
   {
-    this (peer, lcle, true, true);
+    this (peer, locale, false, true);
   }
 
   /**
    * Construct a BeanContextSupport instance.
+   * 
+   * @param peer  the bean context peer (<code>null</code> permitted).
+   * @param locale  the locale (<code>null</code> permitted, equivalent to 
+   *     the default locale).
+   * @param dtime  a flag indicating whether or not the bean context is in
+   *     design time mode.
    */
-  public BeanContextSupport (BeanContext peer, Locale lcle, boolean dtime)
+  public BeanContextSupport (BeanContext peer, Locale locale, boolean dtime)
   {
-    this (peer, lcle, dtime, true);
+    this (peer, locale, dtime, true);
   }
 
   /**
    * Construct a BeanContextSupport instance.
+   * 
+   * @param peer  the bean context peer (<code>null</code> permitted).
+   * @param locale  the locale (<code>null</code> permitted, equivalent to 
+   *     the default locale).
+   * @param dtime  a flag indicating whether or not the bean context is in
+   *     design time mode.
+   * @param visible  initial value of the <code>okToUseGui</code> flag.
    */
-  public BeanContextSupport (BeanContext peer, Locale lcle, boolean dtime,
+  public BeanContextSupport (BeanContext peer, Locale locale, boolean dtime,
                              boolean visible)
   {
     super(peer);
 
-    locale = lcle == null ? Locale.getDefault() : lcle;
+    this.locale = locale == null ? Locale.getDefault() : locale;
     designTime = dtime;
     okToUseGui = visible;