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));