Thank you. Looks good - approved for 7u60. BTW, minor nit (and I might be wrong if this was originally put back to 8 last year):
On the new file, it has: Copyright (c) 2013 Perhaps should be 2014? -jeff On Feb 12, 2014, at 4:14 PM, Christian Thalinger <christian.thalin...@oracle.com> wrote: > On Feb 12, 2014, at 1:06 PM, Jeff Dinkins <jeff.dink...@oracle.com> wrote: > >> >> I might have missed it in the bug, but - where is the review (and who are >> the review approvers?) > > No, you did not miss it; I forgot to paste the link. Here is the review > thread: > > http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-July/010118.html > >> >> thanks, >> >> -jeff >> >> On Feb 12, 2014, at 1:53 PM, Christian Thalinger >> <christian.thalin...@oracle.com> wrote: >> >>> This change was intended to end up in 7u40 but for some reason we missed >>> it. The original bug which went into 8 is: >>> >>> https://bugs.openjdk.java.net/browse/JDK-8019184 >>> >>> and the pushed changeset: >>> >>> http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/bd6949f9dbb2 >>> >>> This is the new 7u60 backport with approval: >>> >>> https://bugs.openjdk.java.net/browse/JDK-8034024 >>> >>> >> >