> > @@ -1229,6 +1397,8 @@ static void print_counters(struct timespec *ts, int > > argc, const char **argv) > > case AGGR_GLOBAL: > > evlist__for_each(evsel_list, counter) > > print_counter_aggr(counter, prefix); > > + if (metric_only) > > + fputc('\n', stat_config.output); > > this new line printing based on metric_only is all over the place.. > could we sorted out new lines in the print callbacks? this makes > my head hurt ;-)
I don't see how a print callback could handle it. It has no idea where the event list ends. Only the high level display function knows that, and it prints the newlines. -Andi -- a...@linux.intel.com -- Speaking for myself only.