[PATCH v2 0/2] perf stat: Fix trailing comma when there is no metric unit

From: James Clark
Date: Wed Oct 30 2024 - 07:40:14 EST


The first commit is failing on Arm and I think the fix should stop more
trailing comma issues which keep happening.

The second one I just noticed when looking at it. I don't feel strongly
about it so not sure if we should do it or not, but seems like the empty
metric-units exclusion from the JSON should be consistent if we're going
to have it at all.

Changes since v1:

* Don't skip printing when the metric-unit string is empty but pass
NULL instead of an empty string.

James Clark (2):
perf stat: Fix trailing comma when there is no metric unit
perf stat: Also hide metric-units from JSON when event didn't run

.../tests/shell/lib/perf_json_output_lint.py | 14 +-
tools/perf/util/stat-display.c | 182 ++++++++++--------
2 files changed, 107 insertions(+), 89 deletions(-)

--
2.34.1