RFR(S): 8170847: Refactor trace/traceStream.hpp

[Markus Gronlund]

Thanks Lois!

Markus

-----Original Message-----
From: Lois Foltan 
Sent: den 7 december 2016 22:40
To: Markus Gronlund
Cc: hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(S): 8170847: Refactor trace/traceStream.hpp


On 12/7/2016 4:37 PM, Lois Foltan wrote:
> Hi Markus,
>
> Looks good.  Can you check for the empty string class loader name at 
> line #75 in traceStream.cpp?

Oops, my mistake, you do check it at line #79.  All looks good then!
Lois

>
> Thanks,
> Lois
>
> On 12/7/2016 9:06 AM, Markus Gronlund wrote:
>> Greetings,
>>
>>
>> Please review refactoring of the traceStream.hpp header into an 
>> associated traceStream.cpp file:
>>
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8170847
>>
>> Webrev: http://cr.openjdk.java.net/~mgronlun/8170847/webrev01/
>>
>>
>> Thanks
>>
>> Markus
>