[7u40] Request for approval: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove

Shi Jun Zhang zhangshj at linux.vnet.ibm.com
Sun Jul 14 22:56:16 PDT 2013


On 7/13/2013 4:27 AM, Seán Coffey wrote:
> Shi Jun,
>
> this is approved for jdk7u40-dev integration. Please push the fix when 
> you get time.
>
> regards,
> Sean.
>
> On 11/07/13 04:06, Shi Jun Zhang wrote:
>> On 7/11/2013 2:18 AM, Seán Coffey wrote:
>>> Shi Jun Zhang,
>>>
>>> Let me get QA approval for this issue and I'll get back to you 
>>> shortly. Is this a must fix request for 7u40 ?
>>>
>>> regards,
>>> Sean.
>>>
>>> On 10/07/13 09:34, Shi Jun Zhang wrote:
>>>> Hi,
>>>>
>>>> I'd like to request for approval to push the following change into 
>>>> 7u40.
>>>>
>>>> Changeset in jdk 8
>>>> http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ed111451b77a
>>>>
>>>> webrev for jdk7u
>>>> http://cr.openjdk.java.net/~zhangshj/jdk7u/8019381/webrev.00/
>>>>
>>>> Reviewed by chegar, mduigou
>>>>
>>>> Review thread
>>>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-June/018450.html 
>>>>
>>>>
>>>> There are some differences between webrev for jdk7u and jdk8. The 
>>>> reason is that some change in jdk8 which invokes isEmpty() is only 
>>>> fixed in jdk8 and not back ported to jdk7u. The jtreg test is 
>>>> modified accordingly.
>>>>
>>>
>> Hi Sean,
>>
>> It's not a must fix for 7u40, I think it's OK to put it in later 
>> release.
>>
>
Thanks, Sean.

I will ask Neil to push the change.

Hi Neil,

Could you help to push the changeset into 7u?

-- 
Regards,

Shi Jun Zhang




More information about the jdk7u-dev mailing list