[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