This patch (committed) reimplements the paramString() method to provide better info, and also updates some API docs in the AccessibleJDesktopPane inner class.

2006-04-20  David Gilbert  <[EMAIL PROTECTED]>

        * javax/swing/JDesktopPane.java
        (paramString): Reimplemented,
        plus API doc updates in AccessibleJDesktopPane.

Regards,

Dave
Index: javax/swing/JDesktopPane.java
===================================================================
RCS file: /sources/classpath/classpath/javax/swing/JDesktopPane.java,v
retrieving revision 1.14
diff -u -r1.14 JDesktopPane.java
--- javax/swing/JDesktopPane.java       5 Apr 2006 15:10:23 -0000       1.14
+++ javax/swing/JDesktopPane.java       20 Apr 2006 11:01:44 -0000
@@ -56,7 +56,6 @@
  */
 public class JDesktopPane extends JLayeredPane implements Accessible
 {
-  /** DOCUMENT ME! */
   private static final long serialVersionUID = 766333777224038726L;
 
   /**
@@ -94,15 +93,15 @@
   private boolean clientDragModeSet = false;
 
   /**
-   * AccessibleJDesktopPane
+   * Provides the accessibility features for the <code>JDesktopPane</code>
+   * component.
    */
   protected class AccessibleJDesktopPane extends AccessibleJComponent
   {
-    /** DOCUMENT ME! */
     private static final long serialVersionUID = 6079388927946077570L;
 
     /**
-     * Constructor AccessibleJDesktopPane
+     * Creates a new <code>AccessibleJDesktopPane</code> instance.
      */
     protected AccessibleJDesktopPane()
     {
@@ -110,9 +109,9 @@
     }
 
     /**
-     * getAccessibleRole
+     * Returns the accessible role for the <code>JSlider</code> component.
      *
-     * @return AccessibleRole
+     * @return [EMAIL PROTECTED] AccessibleRole#DESKTOP_PANE}.
      */
     public AccessibleRole getAccessibleRole()
     {
@@ -298,13 +297,22 @@
   }
 
   /**
-   * This method returns a String that describes the JDesktopPane.
+   * Returns an implementation-dependent string describing the attributes of
+   * this <code>JDesktopPane</code>.
    *
-   * @return A String that describes the JDesktopPane.
+   * @return A string describing the attributes of this 
<code>JDesktopPane</code>
+   *         (never <code>null</code>).
    */
   protected String paramString()
   {
-    return "JDesktopPane";
+    String superParamStr = super.paramString();
+    StringBuffer sb = new StringBuffer();
+    sb.append(",isOptimizedDrawingPossible=");
+    sb.append(isOptimizedDrawingEnabled());
+    sb.append(",desktopManager=");
+    if (desktopManager != null)
+      sb.append(desktopManager);
+    return superParamStr + sb.toString();
   }
 
   /**
@@ -330,9 +338,11 @@
   }
 
   /**
-   * getAccessibleContext
+   * Returns the object that provides accessibility features for this
+   * <code>JDesktopPane</code> component.
    *
-   * @return AccessibleContext
+   * @return The accessible context (an instance of 
+   *     [EMAIL PROTECTED] AccessibleJDesktopPane}).
    */
   public AccessibleContext getAccessibleContext()
   {

Reply via email to