From: Duy Nguyen <pclouds@gmail.com>
Date: Tue, 03 Jan 2017 12:14:25 GMT
Subject: Re: [PATCH v3 08/16] pathspec: always show mnemonic and name in unsupported_magic
Message-ID: <CACsJy8Cja1um2oFDWufyk_7xaZbf9+=kyuWFx==Vb0nHiMqiwA@mail.gmail.com>
In-Reply-To: <1481670870-66754-9-git-send-email-bmwill@google.com>

On Wed, Dec 14, 2016 at 6:14 AM, Brandon Williams <bmwill@google.com> wrote:
> @@ -340,8 +336,9 @@ static void NORETURN unsupported_magic(const char *pattern,
>                         continue;
>                 if (sb.len)
>                         strbuf_addch(&sb, ' ');
> -               if (short_magic & m->bit)
> -                       strbuf_addf(&sb, "'%c'", m->mnemonic);
> +
> +               if (m->mnemonic)
> +                       strbuf_addf(&sb, "'(%c)%s'", m->mnemonic, m->name);
>                 else
>                         strbuf_addf(&sb, "'%s'", m->name);
>         }

The die() call is out of diff context, but it'll print

pathspec magic not supported by this command: (!)top

which looks too much like :(<name>)<mnemonic> pathspec syntax too me
and threw me off a bit. And it's a bit cryptic, isn't it? Since this
is meant for human, maybe we can just write

pathspec magic not supported by this command: top (mnemonic: '!')
-- 
Duy