2020-08-14 10:49:03

by Peng Fan

[permalink] [raw]
Subject: [PATCH] perf bench numa: Remove dead code in parse_nodes_opt()

In the function parse_nodes_opt(), the statement "return 0;" is dead
code, remove it.

Signed-off-by: Peng Fan <[email protected]>
---
tools/perf/bench/numa.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/tools/perf/bench/numa.c b/tools/perf/bench/numa.c
index 5797253..70c8c2c 100644
--- a/tools/perf/bench/numa.c
+++ b/tools/perf/bench/numa.c
@@ -729,8 +729,6 @@ static int parse_nodes_opt(const struct option *opt __maybe_unused,
return -1;

return parse_node_list(arg);
-
- return 0;
}

#define BIT(x) (1ul << x)
--
2.1.0


2020-08-14 13:37:52

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: Re: [PATCH] perf bench numa: Remove dead code in parse_nodes_opt()

Em Fri, Aug 14, 2020 at 06:44:54PM +0800, Peng Fan escreveu:
> In the function parse_nodes_opt(), the statement "return 0;" is dead
> code, remove it.

Thanks, applied.

- Arnaldo

> Signed-off-by: Peng Fan <[email protected]>
> ---
> tools/perf/bench/numa.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/tools/perf/bench/numa.c b/tools/perf/bench/numa.c
> index 5797253..70c8c2c 100644
> --- a/tools/perf/bench/numa.c
> +++ b/tools/perf/bench/numa.c
> @@ -729,8 +729,6 @@ static int parse_nodes_opt(const struct option *opt __maybe_unused,
> return -1;
>
> return parse_node_list(arg);
> -
> - return 0;
> }
>
> #define BIT(x) (1ul << x)
> --
> 2.1.0
>

--

- Arnaldo