RFR: 8202889: Remove trailing LF from perf log
Yasumasa Suenaga
yasuenag at gmail.com
Thu May 10 05:42:58 UTC 2018
Hi Thomas,
Thank you for your review!
I will update copyright year in all files.
I'm waiting for second reviewer.
Thanks,
Yasumasa
2018-05-10 14:19 GMT+09:00 Thomas Stüfe <thomas.stuefe at gmail.com>:
> Hi Yasumasa,
>
> seems fine and trivial. Please update copyright years (in case of aix
> specific files, please both SAP and Oracle). I do not need a new
> webrev.
>
> Thanks, Thomas
>
> On Thu, May 10, 2018 at 6:13 AM, Yasumasa Suenaga <yasuenag at gmail.com> wrote:
>> Hi all,
>>
>> Could you review this change?
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8202889
>> webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8202889/webrev.00/
>>
>>
>> JDK-8168122 introduced new UL tags for perfMemory. However they trails
>> unnecessary LF in each log entries.
>>
>> Examples are here:
>> http://cr.openjdk.java.net/~ysuenaga/openjdk-fc28/trace-all.log
>>
>> This change removes unnecessary LFs.
>>
>>
>> Thanks,
>>
>> Yasumasa
More information about the hotspot-runtime-dev
mailing list