Hey Neil,

I wouldn't regard this as a risky fix and don't see a problem with it going into jdk7u-dev now (before phase2 approval process begins in mid September) It also benefits from having a unit testcase. Testing (jtreg tests) in the jdk8/tl repos would have occurred on a regular basis also since the fix was push there.

Alan was initial reviewer and may wish to comment (cc'ed)

regards,
Sean.

On 04/09/2012 13:30, Neil Richards wrote:
On Mon, 2012-09-03 at 09:58 +0100, Seán Coffey wrote:
Approved for jdk7u-dev. (7u10)

I've created a bug record for this and will mark as fixed once I see
your push.
cc'ing initial reviewer also.

regards,
Sean.

On 03/09/2012 05:18, Jonathan Lu wrote:
Hi folks,

Here's a request of porting the fix of 7190219 back to JDK7, could you
please review it?

Bug: http://bugs.sun.com/view_bug.do?bug_id=7190219

Change set: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bf0c6f91bc22

Thread where it was reviewed:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-August/011087.html


Thanks,
Jonathan

Hi Sean,
It looks like this changeset has not yet been promoted from the
component (tl) forest to the master jdk8 repository.

Please confirm, in light of this, if you'd like this change to be
applied to jdk7u at this point, or if you would like it to spend some
(more) time being proved in the jdk8 stream prior to this happening.

Regards,
Neil


Reply via email to