java.awt.MenuItem confused the javac 1.3 compiler. I added an explicit this. and removed some duplicate code.

2005-07-27  Roman Kennke  <[EMAIL PROTECTED]>

        * java/awt/MenuItem.java
        (AccessibleAWTMenuItem.setCurrentAccessibleValue): Removed
        duplicate code. Added this to make setEnabled unambiuous for
        javac 1.3 compiler.

/Roman
Index: java/awt/MenuItem.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/awt/MenuItem.java,v
retrieving revision 1.24
diff -u -r1.24 MenuItem.java
--- java/awt/MenuItem.java      2 Jul 2005 20:32:25 -0000       1.24
+++ java/awt/MenuItem.java      27 Jul 2005 08:51:17 -0000
@@ -179,14 +179,11 @@
      */
     public boolean setCurrentAccessibleValue(Number number)
     {
-      if (number.intValue() == 0)
-       {
-         setEnabled(false);
-         return false;
-       }
-    
-      setEnabled(true);
-      return true;
+      boolean result = (number.intValue() != 0);
+      // this. is required by javac 1.3, otherwise it is confused with
+      // MenuItem.this.setEnabled.
+      this.setEnabled(result);
+      return result; 
     }
 
     /* (non-Javadoc)
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to