RFR: 8152637: Create a stack allocatable LogStream class

Robbin Ehn robbin.ehn at oracle.com
Mon Apr 4 15:15:47 UTC 2016


Hi Stefan,

Looks even better, thanks!

/Robbin

On 04/04/2016 02:05 PM, Stefan Karlsson wrote:
> Hi all,
>
> I've updated the patch to use LogTagSets instead of function pointers:
>
>   http://cr.openjdk.java.net/~stefank/8152637/webrev.02.delta
>   http://cr.openjdk.java.net/~stefank/8152637/webrev.02
>
> The patch is rebased against the LogTagSet enhancements in:
>
>   http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-April/022368.html
>
> I've also updated the unit tests after feedback on earlier UL patches.
>
> Thanks,
> StefanK
>
> On 2016-03-29 15:47, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this patch to create a stack allocatable LogStream class
>> that embeds the ResourceMark.
>>
>> http://cr.openjdk.java.net/~stefank/8152637/webrev.01
>> https://bugs.openjdk.java.net/browse/JDK-815263
>>
>> The patch is applied on top of the patch in:
>> http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-March/022303.html
>>
>> I've converted the code in binaryTreeDictionary.cpp to show how to use
>> the class.
>>
>> Test: new internal vm test, jprt
>>
>> Thanks,
>> StefanK
>


More information about the hotspot-dev mailing list