Re: RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-07 Thread Shi Jun Zhang
On 7/5/2013 10:58 AM, Jonathan Lu wrote: On 07/03/2013 11:04 AM, Shi Jun Zhang wrote: On 7/1/2013 11:49 PM, Chris Hegarty wrote: On 1 Jul 2013, at 17:22, Remi Forax fo...@univ-mlv.fr wrote: On 07/01/2013 09:43 AM, Shi Jun Zhang wrote: On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: On

Re: RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-04 Thread Jonathan Lu
On 07/03/2013 11:04 AM, Shi Jun Zhang wrote: On 7/1/2013 11:49 PM, Chris Hegarty wrote: On 1 Jul 2013, at 17:22, Remi Forax fo...@univ-mlv.fr wrote: On 07/01/2013 09:43 AM, Shi Jun Zhang wrote: On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: On 6/28/2013 9:02 PM, Alan Bateman wrote: On

Re: RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-02 Thread Shi Jun Zhang
On 7/1/2013 11:49 PM, Chris Hegarty wrote: On 1 Jul 2013, at 17:22, Remi Forax fo...@univ-mlv.fr wrote: On 07/01/2013 09:43 AM, Shi Jun Zhang wrote: On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: On 6/28/2013 9:02 PM, Alan Bateman wrote: On 27/06/2013 22:13, Remi Forax wrote: On 06/27/2013

RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-01 Thread Shi Jun Zhang
On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: 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.

Re: RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-01 Thread Remi Forax
On 07/01/2013 09:43 AM, Shi Jun Zhang wrote: On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: 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

Re: RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-01 Thread Chris Hegarty
On 1 Jul 2013, at 17:22, Remi Forax fo...@univ-mlv.fr wrote: On 07/01/2013 09:43 AM, Shi Jun Zhang wrote: On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: 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

Re: RFR: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

2013-07-01 Thread Mike Duigou
+1 On 2013-07-01, at 1:43, Shi Jun Zhang zhang...@linux.vnet.ibm.com wrote: On 6/29/2013 12:05 AM, Shi Jun Zhang wrote: 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