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

[Neil Richards]

Now pushed to jdk7u-dev/jdk [1].

Regards,
Neil

[1] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/b0b15e373002


On Mon, 2013-07-15 at 13:56 +0800, Shi Jun Zhang wrote:
> 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?
> 

-- 
Unless stated above:
IBM email: neil_richards at uk.ibm.com
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU