Bruno Haible <[EMAIL PROTECTED]> writes:

> !   error (0, 0,
> !      (problem == -1
> !       ? _("invalid argument %s for %s")
> !       : _("ambiguous argument %s for %s")),
> !      quotearg_n_style (0, ARGMATCH_QUOTING_STYLE, value),
>        quote_n (1, context));

That looks good, thanks, but can you please hoist the call to gettext
outside the conditional?  That is a tad easier for me to read and
typically generates more-compact code.  Something like this:

  error (0, 0,
         _(problem == -1
           ? "invalid argument %s for %s"
           : "ambiguous argument %s for %s"),
         quotearg_n_style (0, ARGMATCH_QUOTING_STYLE, value),
         quote_n (1, context));


Reply via email to