Re: [PATCH 6/7] perf hists browser: Split popup menu actions

[Jiri Olsa <jolsa@redhat.com>]

On Sun, Apr 19, 2015 at 01:04:14PM +0900, Namhyung Kim wrote:

SNIP

> +
> +static int
> +add_script_opt(struct popup_option *opt, char **optstr,
> +	       struct hist_browser *browser __maybe_unused,
> +	       struct thread *thread, struct symbol *sym)
> +{
> +	if (thread) {
> +		if (asprintf(optstr, "Run scripts for samples of thread [%s]",
> +			     thread__comm_str(thread)) < 0)
> +			return 0;
> +	} else if (sym) {

there's also check for sym->namelen in the original code

> +		if (asprintf(optstr, "Run scripts for samples of symbol [%s]",
> +			     sym->name) < 0)
> +			return 0;
> +	} else {
> +		if (asprintf(optstr, "Run scripts for all samples") < 0)
> +			return 0;
> +	}
> +
> +	opt->thread = thread;
> +	opt->sym = sym;
> +	opt->fn = do_run_script;
> +
> +	return 1;
> +}

SNIP