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

James Clark posted 2 patches 3 weeks, 4 days ago
There is a newer version of this series
.../tests/shell/lib/perf_json_output_lint.py  |  14 +-
tools/perf/util/stat-display.c                | 182 ++++++++++--------
2 files changed, 107 insertions(+), 89 deletions(-)
[PATCH v2 0/2] perf stat: Fix trailing comma when there is no metric unit
Posted by James Clark 3 weeks, 4 days ago
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