Static checkers complain about the unused assignment to pfx_len. The code can obviously be simplified.
Signed-off-by: Jeff Moyer <jmo...@redhat.com> --- util/abspath.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/util/abspath.c b/util/abspath.c index 09bbd27..e44236f 100644 --- a/util/abspath.c +++ b/util/abspath.c @@ -9,11 +9,7 @@ char *prefix_filename(const char *pfx, const char *arg) struct strbuf path = STRBUF_INIT; size_t pfx_len = pfx ? strlen(pfx) : 0; - if (!pfx_len) - ; - else if (is_absolute_path(arg)) - pfx_len = 0; - else + if (pfx_len && !is_absolute_path(arg)) strbuf_add(&path, pfx, pfx_len); strbuf_addstr(&path, arg); -- 2.19.1 _______________________________________________ Linux-nvdimm mailing list -- linux-nvdimm@lists.01.org To unsubscribe send an email to linux-nvdimm-le...@lists.01.org