RFR: 8202889: Remove trailing LF from perf log

[Langer, Christoph]

+1

> -----Original Message-----
> From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-
> bounces at openjdk.java.net] On Behalf Of Yasumasa Suenaga
> Sent: Donnerstag, 10. Mai 2018 07:43
> To: Thomas Stüfe <thomas.stuefe at gmail.com>
> Cc: Hotspot dev runtime <hotspot-runtime-dev at openjdk.java.net>
> Subject: Re: RFR: 8202889: Remove trailing LF from perf log
> 
> 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