PatchSet 4173 
Date: 2003/11/28 16:15:16
Author: dalibor
Branch: HEAD
Tag: (none) 
Log:
Fixed mauve failure for Hashtable

Members: 
        ChangeLog:1.1765->1.1766 
        libraries/javalib/java/util/Hashtable.java:1.30->1.31 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.1765 kaffe/ChangeLog:1.1766
--- kaffe/ChangeLog:1.1765      Thu Nov 27 18:27:12 2003
+++ kaffe/ChangeLog     Fri Nov 28 16:15:16 2003
@@ -1,3 +1,16 @@
+2003-11-28  Dalibor Topic <[EMAIL PROTECTED]>
+
+       Reported by: Mark Wielaard  <[EMAIL PROTECTED]>
+       
+       * libraries/javalib/java/util/Hashtable.java
+       (contains): Improved comment.
+
+       Reported by: Jeroen Frijters  <[EMAIL PROTECTED]>
+
+       * libraries/javalib/java/util/Hashtable.java
+       (containsValue): Delegate to contains(Object) to make sure older
+       code overwriting it continues to work.
+       
 2003-11-27  Guilhem Lavaux <[EMAIL PROTECTED]>
 
        * libraries/javalib/kjc.jar: Regenerated.
Index: kaffe/libraries/javalib/java/util/Hashtable.java
diff -u kaffe/libraries/javalib/java/util/Hashtable.java:1.30 
kaffe/libraries/javalib/java/util/Hashtable.java:1.31
--- kaffe/libraries/javalib/java/util/Hashtable.java:1.30       Sun Oct  5 18:25:11 
2003
+++ kaffe/libraries/javalib/java/util/Hashtable.java    Fri Nov 28 16:15:18 2003
@@ -334,7 +334,8 @@
   public synchronized boolean contains(Object value)
   {
     /* delegate to non-overridable worker method 
-     * to avoid blowing up the stack.
+     * to avoid blowing up the stack, when called 
+     * from overridden contains[Value]() method.
      */
     return internalContainsValue(value);
   }
@@ -353,10 +354,10 @@
    */
   public boolean containsValue(Object value)
   {
-    /* delegate to non-overridable worker method 
-     * to avoid blowing up the stack.
+    /* delegate to older method to make sure code overwriting it 
+     * continues to work.
      */
-    return internalContainsValue(value);
+    return contains(value);
   }
 
   /**

_______________________________________________
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe

Reply via email to