perf annotate: Remove redundant 'ret' variable

Return the result from hist_entry_iter__add() directly instead of taking
this in another redundant variable.

Signed-off-by: tangmeng <tangmeng@uniontech.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/20220216030425.27779-1-tangmeng@uniontech.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c
index 5e038d9bab058cbb2a7b7662526a151e67ff6307..e65dc380be15842bde8ea387fc4e8097bc6f18d3 100644 (file)
--- a/tools/perf/builtin-annotate.c
+++ b/tools/perf/builtin-annotate.c
@@ -194,7 +194,6 @@ static int process_branch_callback(struct evsel *evsel,
        };
 
        struct addr_location a;
-       int ret;
 
        if (machine__resolve(machine, &a, sample) < 0)
                return -1;
@@ -207,8 +206,7 @@ static int process_branch_callback(struct evsel *evsel,
 
        hist__account_cycles(sample->branch_stack, al, sample, false, NULL);
 
-       ret = hist_entry_iter__add(&iter, &a, PERF_MAX_STACK_DEPTH, ann);
-       return ret;
+       return hist_entry_iter__add(&iter, &a, PERF_MAX_STACK_DEPTH, ann);
 }
 
 static bool has_annotation(struct perf_annotate *ann)