Mark Wielaard wrote:
Hi,
On Fri, 2003-11-28 at 17:39, Dalibor Topic wrote:
Thanks for pointing that mistake out. I've attached an updated version of the patch.
2003-11-28 Dalibor Topic <[EMAIL PROTECTED]>
Reported by: Jim Pick <[EMAIL PROTECTED]>
* libraries/javalib/java/util/Hashtable.java (internalcontainsValue): New method. (contains) Delegate to internalContainsValue.
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.
I'm not sure how to deal with mutliple people's bug reports being fixed in this patch in the ChangeLog entry, so I'd appreciate a hint from the ChangeLog police.
Looks fine to me. (I would loose the extra empty line between Reported by: and the actual change, but feel free to ignore me).
Also you could now pull the complete internalContainsValue() into the containsValue() method, but that would make the patch bigger, so don't bother (compiler/VM will inline the private method anyway).
O.K.
If this solves both Mauve Hashtable tests, please check it in.
done.
cheers, dalibor topic
_______________________________________________ Classpath mailing list [EMAIL PROTECTED] http://mail.gnu.org/mailman/listinfo/classpath
