RFR: 8202889: Remove trailing LF from perf log

[Yasumasa Suenaga]

Thanks!

2018年5月11日(金) 6:04 Langer, Christoph <christoph.langer at sap.com>:

> +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
>