Request for approval: 7181353: Update error message to distinguish native OOM and java OOM in net

[Neil Richards]

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

Regards,
Neil

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

On Tue, 2012-09-25 at 13:17 +0100, Seán Coffey wrote:
> Approved for jdk7u-dev then.  I'll update bug records when I see the push.
> 
> > Please don't log a new CR, I think it is not necessary. Thanks.
> If you ever want to update the message to be more precise, just mail me 
> and I'll get a bug record created.
> 
> regards,
> Sean.
> 
> On 25/09/2012 08:00, Sean Chou wrote:
> > Hi Seán,
> >
> > I updated the webrev, now the message is the same as in jdk8.
> >
> > New webrev:  http://cr.openjdk.java.net/~zhouyx/7181353/jdk7u/webrev.01/
> >
> > Please don't log a new CR, I think it is not necessary. Thanks.
> >
> > On Mon, Sep 24, 2012 at 6:59 PM, Seán Coffey <sean.coffey at oracle.com> wrote:
> >> Sean,
> >>
> >> tweaking the error messages makes more sense and helps diagnostics. Could I
> >> ask you to port the original  7181353 changeset from jdk8 to 7u ? I can log
> >> a new CR where you could then update jdk8 & 7u with the modified strings if
> >> you wish. I think it makes sense to keep 7u and 8 consistent in this area.
> >>
> >> What do you think ?
> >>
> >> Regards,
> >> Sean.
> >>
> >>
> >> On 24/09/12 08:16, Sean Chou wrote:
> >>> Hello,
> >>>
> >>> This is request for approval to push the fix of 7181353 to jdk7u,
> >>> please take a look.
> >>>
> >>> Link to the bug:
> >>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7181353
> >>>
> >>> Changeset in jdk8: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/15a6b0bceb1e
> >>>
> >>> Webrev for jdk7u: http://cr.openjdk.java.net/~zhouyx/7181353/jdk7u/webrev/
> >>>
> >>> There are several differences between changeset in jdk8 and webrev for
> >>> jdk7,
> >>> however, all modifications are about adding a little more information to
> >>> error
> >>> message.
> >>>
> >>> Review thread:
> >>> http://mail.openjdk.java.net/pipermail/net-dev/2012-July/004516.html
> >>>
> >
> >
> 


-- 
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