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

Reply via email to