diff options
| author |   Zheng Yejian <zhengyejian1@huawei.com> | 2022-06-30 09:31:52 +0800 |
|---|---|---|
| committer |   Steven Rostedt (Google) <rostedt@goodmis.org> | 2022-07-12 17:35:11 -0400 |
| commit | fb991f1942334b0cbf6aa6a88faa586ba22d3550 (patch) | |
| tree | 13472bcbcfa23c7773b50771a4006b6675bbf30b /include/linux/timerqueue.h | |
| parent | tracing/user_events: Fix syntax errors in comments (diff) | |
| download | linux-dev-fb991f1942334b0cbf6aa6a88faa586ba22d3550.tar.xz linux-dev-fb991f1942334b0cbf6aa6a88faa586ba22d3550.zip | |
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/20220630013152.164871-1-zhengyejian1@huawei.com
Signed-off-by: Zheng Yejian <zhengyejian1@huawei.com>
Reviewed-by: Tom Rix <trix@redhat.com>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Diffstat (limited to 'include/linux/timerqueue.h')
0 files changed, 0 insertions, 0 deletions