On 4 December 2015 at 17:53, Jakub Jelinek <ja...@redhat.com> wrote:
> +
> +                 if (e->unknown_error)
> +                   error_at (loc, e->unknown_error, option->opt_text);
> +                 else
> +                   error_at (loc, "unrecognized argument in option %qs",
> +                             option->opt_text);

The same code that handles command-line options has:

      if (e->unknown_error)
    error_at (loc, e->unknown_error, decoded->arg);
      else
    error_at (loc, "unrecognized argument in option %qs", opt);

My guess is that the first error_at should use arg instead of
option->opt_text to be equivalent. Of course, ideally, this code would
not be duplicated, but rather merged "somehow".

Cheers,

Manuel.

Reply via email to