On Wed, Apr 12, 2017 at 06:21:05AM +0800, Jin Yao wrote: SNIP
> +static int hist_iter__branch_callback(struct hist_entry_iter *iter,
> + struct addr_location *al __maybe_unused,
> + bool single __maybe_unused,
> + void *arg)
> +{
> + struct hist_entry *he = iter->he;
> + struct report *rep = arg;
> + struct branch_info *bi;
> +
> + if (sort__mode == SORT_MODE__BRANCH) {
is this check necessary? the hist_iter__branch_callback
was set based on this check
jirka

