RFR: 8061219: Implement unit-tests for UL

Coleen Phillimore coleen.phillimore at oracle.com
Fri Jul 22 15:37:59 UTC 2016


Hi Marcus,

This looks good.  Thank you for adding the comments.

Coleen


On 7/20/16 6:55 AM, Marcus Larsson wrote:
> Hi again,
>
> Updated the patch with additional comments. Also fixed the copyright 
> header.
>
> New webrev:
> http://cr.openjdk.java.net/~mlarsson/8061219/webrev.01/
> Incremental:
> http://cr.openjdk.java.net/~mlarsson/8061219/webrev.00-01/
>
> Thanks,
> Marcus
>
>
> On 2016-07-19 10:25, Marcus Larsson wrote:
>> Still looking for a *R*eviewer for this.
>>
>> Thanks,
>> Marcus
>>
>>
>> On 07/18/2016 10:49 AM, Marcus Larsson wrote:
>>> Hi,
>>>
>>> Please review the following patch to add unit tests for the unified 
>>> logging framework. Tests are written for the new gtest unit test 
>>> framework. Most of the public functions for UL should be covered, 
>>> but later additions have internal vm tests (in log.cpp) that will be 
>>> ported separately.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~mlarsson/8061219/webrev.00/
>>>
>>> Issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8061219
>>>
>>> Testing:
>>> New tests through JPRT
>>>
>>> Thanks,
>>> Marcus
>>
>



More information about the hotspot-runtime-dev mailing list