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 <[email protected]> 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 >> > -- Best Regards, Sean Chou
