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