[7u40] Request for approval: 8019381: HashMap.isEmpty is non-final, potential issues for get/remove
Shi Jun Zhang
zhangshj at linux.vnet.ibm.com
Wed Jul 10 20:06:13 PDT 2013
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.
--
Regards,
Shi Jun Zhang
More information about the jdk7u-dev
mailing list