Re: [PATCH 01/14] perf tool: Introduce perf_hpp__list for period related columns

Namhyung Kim
2012-11-29T10:52:23Z

Hi,

On Thu, 29 Nov 2012 16:44:59 +0900, Namhyung Kim wrote:
> Hi Jiri,
>
> On Wed, 28 Nov 2012 14:52:36 +0100, Jiri Olsa wrote:
>> Adding perf_hpp__list list to register and contain all period
>> related columns the command is interested in.
>>
>> This way we get rid of static array holding all possible
>> columns and enable commands to register their own columns.
>
> But it seems you didn't get rid of the array? :)

Ah, understood. You wanted to add new hpp columns dynamically!
Looks good to me then.

Acked-by: Namhyung Kim <namhyung@kernel.org>

Thanks,
Namhyung