matches() checks whether its first argument is a prefix of its second argument. Therefore, matching "-human-readable" already covers "-human" and its abbreviations. Remove the redundant condition. Signed-off-by: Mohammad abu saleh --- ip/ip.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ip/ip.c b/ip/ip.c index e4b71bde..dd84f94b 100644 --- a/ip/ip.c +++ b/ip/ip.c @@ -224,8 +224,7 @@ int main(int argc, char **argv) preferred_family = AF_MPLS; } else if (strcmp(opt, "-B") == 0) { preferred_family = AF_BRIDGE; - } else if (matches(opt, "-human") == 0 || - matches(opt, "-human-readable") == 0) { + } else if (matches(opt, "-human-readable") == 0) { ++human_readable; } else if (matches(opt, "-iec") == 0) { ++use_iec; -- 2.53.0