diff options
| author | Zheng Yejian <[email protected]> | 2022-06-30 09:31:52 +0800 |
|---|---|---|
| committer | Steven Rostedt (Google) <[email protected]> | 2022-07-12 17:35:11 -0400 |
| commit | fb991f1942334b0cbf6aa6a88faa586ba22d3550 (patch) | |
| tree | 13472bcbcfa23c7773b50771a4006b6675bbf30b /tools/perf/scripts/python/export-to-sqlite.py | |
| parent | 94c255ac676fa922df3498608ead42b0a0c85122 (diff) | |
tracing/histograms: Simplify create_hist_fields()
When I look into implements of create_hist_fields(), I think there can be
following two simplifications:
1. If something wrong happened in parse_var_defs(), free_var_defs() would
have been called in it, so no need goto free again after calling it;
2. After calling create_key_fields(), regardless of the value of 'ret', it
then always runs into 'out: ', so the judge of 'ret' is redundant.
Link: https://lkml.kernel.org/r/[email protected]
Signed-off-by: Zheng Yejian <[email protected]>
Reviewed-by: Tom Rix <[email protected]>
Signed-off-by: Steven Rostedt (Google) <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-sqlite.py')
0 files changed, 0 insertions, 0 deletions