[cp-patches] RFA: Canvas.java

[Tania Bento]

Hey,

This patch fixes a Harmony Test that was failing on Classpath.  We now
generate the canvas name in the same manner as Sun.  If someone could
please approve/comment on this patch, that would be great. 

2006-07-06  Tania Bento  <[EMAIL PROTECTED]>

        * gnu/java/awt/Canvas.java
        Added new private variable 'next_canvas_number".
        (generateName): Added.
        (getUniqueLong): Added.

Thanks,
Tania

Index: java/awt/Canvas.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/awt/Canvas.java,v
retrieving revision 1.16
diff -u -r1.16 Canvas.java
--- java/awt/Canvas.java	27 Sep 2005 18:27:45 -0000	1.16
+++ java/awt/Canvas.java	6 Jul 2006 16:15:27 -0000
@@ -68,6 +68,11 @@
    * Compatible with Sun's JDK.
    */
   private static final long serialVersionUID = -2284879212465893870L;
+  
+  /**
+   * The number used to generate the name returned by getName.
+   */
+  private static transient long next_canvas_number;
 
   /**
    * The graphics configuration associated with the canvas.
@@ -343,4 +348,19 @@
     /* Call the paint method */
     paint(graphics);
   }
+  
+  /**
+   * Generate a unique name for this canvas.
+   *
+   * @return A unique name for this canvas.
+   */
+  String generateName()
+  {
+    return "canvas" + getUniqueLong();
+  }
+
+  private static synchronized long getUniqueLong()
+  {
+    return next_canvas_number++;
+  }
 }