From 1e2b38ba784eff9a7d50cfca990cdf4d63687913 Mon Sep 17 00:00:00 2001 From: yidong0635 Date: Thu, 22 Oct 2020 14:37:39 -0400 Subject: [PATCH] example/perf: Fix warning about scanbuild. rc has been assigned to -1 directly, so it's no used here. Issue: accel_perf.c:681:7: warning: Although the value stored to 'rc' is used in the enclosing expression, the value is never actually read from 'rc' if ((rc = spdk_app_parse_args(argc, argv, &opts, "o:q:t:yw:P:f:", NULL, parse_args, ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. Signed-off-by: yidong0635 Change-Id: I1d0a4447c4655bde97fa18d50b694809fc9010ea Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4825 Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk Reviewed-by: Shuhei Matsumoto Reviewed-by: Tomasz Zawadzki Reviewed-by: Jim Harris Reviewed-by: Paul Luse --- examples/accel/perf/accel_perf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/accel/perf/accel_perf.c b/examples/accel/perf/accel_perf.c index 2093253c1..899a2b63c 100644 --- a/examples/accel/perf/accel_perf.c +++ b/examples/accel/perf/accel_perf.c @@ -678,8 +678,8 @@ main(int argc, char **argv) pthread_mutex_init(&g_workers_lock, NULL); spdk_app_opts_init(&opts); opts.reactor_mask = "0x1"; - if ((rc = spdk_app_parse_args(argc, argv, &opts, "o:q:t:yw:P:f:", NULL, parse_args, - usage)) != SPDK_APP_PARSE_ARGS_SUCCESS) { + if (spdk_app_parse_args(argc, argv, &opts, "o:q:t:yw:P:f:", NULL, parse_args, + usage) != SPDK_APP_PARSE_ARGS_SUCCESS) { rc = -1; goto cleanup; }