Question on HashMap change in 8011200 (original) (raw)
Shi Jun Zhang zhangshj at linux.vnet.ibm.com
Fri Jun 28 16:05:39 UTC 2013
- Previous message: Question on HashMap change in 8011200
- Next message: hg: jdk8/tl/langtools: 8014513: Sjavac doesn't detect 32-bit jvm properly
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 6/28/2013 9:02 PM, Alan Bateman wrote:
On 27/06/2013 22:13, Remi Forax wrote:
On 06/27/2013 10:02 AM, Shi Jun Zhang wrote:
Hi,
There are some isEmpty() check added into get/remove methods since 8011200 to return directly if HashMap is empty. However isEmpty is a non-final public method which can be overridden by subclass. If the subclass defines isEmpty differently from HashMap, it would cause problem while getting or removing elements. yes, it's a bug. Could you report it ? Rémi I've created a bug to track this: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove -Alan Thanks, Alan.
I'm quite busy today and do not have time to report it until now. Thanks for your help.
I will provide a webrev next Monday for review.
-- Regards,
Shi Jun Zhang
- Previous message: Question on HashMap change in 8011200
- Next message: hg: jdk8/tl/langtools: 8014513: Sjavac doesn't detect 32-bit jvm properly
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]