JVM/TI code review request (XS and M) (7182152)

Daniel D. Daugherty daniel.daugherty at oracle.com
Wed Feb 13 13:08:12 PST 2013


On 2/13/13 1:54 PM, Alan Bateman wrote:
> On 12/02/2013 00:41, Daniel D. Daugherty wrote:
>> Greetings,
>>
>> Right after I pushed the changeset for 8007420, Alan B let me know that
>> he had updated the JLI/JPLIS test infrastructure to support running the
>> tests with a JRE instead of a JDK. Alan made his changes in early 
>> January,
>> but I had developed this test back in December. I never checked to see
>> if the test on which I based my new test had changed. Sigh...
>>
>> I filed a new bug to update the new test. Here is the webrev URL:
>>
>> http://cr.openjdk.java.net/~dcubed/8007935-webrev/0-jdk8-tl/
>>
>> As always, comments and suggestions are welcome.
>>
>> Dan
> Looks good to me too, and sorry I didn't spot this on the initial review.
>
> -Alan

Thanks for the review, Alan!

I'll be pushing the fix shortly...

Dan



More information about the serviceability-dev mailing list