On Tue, 16 Jun 2020 09:39:02 +0300
Anton Danilov <littlesmilingcl...@gmail.com> 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.

Reply via email to