Re: [PATCH 3/6] perf script: move printing of 'common' data from print_event and rename

Steven Rostedt
2011-03-10T17:07:13Z

On Thu, 2011-03-10 at 09:45 -0700, David Ahern wrote:
>
> On 03/10/11 09:20, Arnaldo Carvalho de Melo wrote:
> >>> I think the merge with the libraries you mention is something that is
> >>> not a precondition to merging David's work.
> >>>
> >>> And his work is at least moving things in the trace parts of tools/,
> >>> that was so far dormant.
> >>
> >> Note, I'm not saying that his work should be put on hold. I actually
> >> want to start it up again. Maybe I worded this response wrong. It was
> >> more about bringing David up to speed on what I want to do next than
> >> saying not to do this work.
> >
> > Right, I just wanted to make sure he doesn't spill his coffee with what
> > could seem like yet another requirement to work on infrastructure to get
> > his legitimate feature implementation merged :-)
>
> I appreciate the clarification. Luckily, no coffee in hand.
>
> I need to get the time-of-day strings done next, and someone else is
> pushing the reverse callchain display (another local change I wanted to
> get upstream). I could take a look at other requests - like common code
> - as time allows.

Right, we could start looking at making the common code on top of your
changes. I don't want to hinder progress for asking for too much.

Thanks,

-- Steve