Re: [PATCH 4/5] test-tool: test refspec input/output

From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Apr 7, 2021, 08:54 UTC
Message-ID: <87lf9ujwym.fsf@evledraar.gmail.com>
In-Reply-To: <c8d1de06f84499f2f56b3a06df665630806f94ce.1617627856.git.gitgitgadget@gmail.com>

On Mon, Apr 05 2021, Derrick Stolee via GitGitGadget wrote:

Show 15 quoted lines

> +static const char * const refspec_usage[] = {
> +	N_("test-tool refspec [--fetch]"),
> +	NULL
> +};
> +
> +int cmd__refspec(int argc, const char **argv)
> +{
> +	struct strbuf line = STRBUF_INIT;
> +	int fetch = 0;
> +
> +	struct option refspec_options [] = {
> +		OPT_BOOL(0, "fetch", &fetch,
> +			 N_("enable the 'fetch' option for parsing refpecs")),
> +		OPT_END()
> +	};

I don't think we should waste translator time by marking these (or anything else in t/helper) with N_()).

I see you probably copied this from elsewhere & we should probably fix the existing ones, but no reason to add new ones...