Re: [PATCH 4/8] perf, tools, stat: Implement CSV metrics output

From: Andi Kleen
Date: Tue Feb 02 2016 - 09:03:12 EST


On Tue, Feb 02, 2016 at 10:50:20AM +0100, Jiri Olsa wrote:
> > Two new fields are added: metric value and metric name before
> > the running/enabled fields.
>
> I see, I thought it was meant to be the other way around,
> so I need to ask now, wouldn't this break existing parsers?

It would, but the new fields weren't around for long and I don't think there
are many users so it's probably ok
(I have a tool that uses them and I'm ok with updating it)

-Andi

--
ak@xxxxxxxxxxxxxxx -- Speaking for myself only