Subject: Re: [PATCH v2] tc: qdisc: filter qdisc's by parent/handle specification

From: Stephen Hemminger <[email-masked]>
Date: Thu, 18 Jun 2020 15:17:02 -0700

On Tue, 16 Jun 2020 09:39:02 +0300
Anton Danilov <[email-masked]> wrote:

> +				fprintf(stderr,
> +					"only one of parent/handle/root/ingress can be specified\n");
> +				arg_error = true;
> +				break;

The concept is good, but it could be simplified.

You are repeating same error message several places and it is not
necessary to continue parsing after one bad argument.

See what invarg() does.