Re: [PATCH v3][Outreachy] branch -D: allow - as abbreviation of @{-1}

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



elena petrashen <elena.petrashen@xxxxxxxxx> wrote:
> On Thu, Mar 31, 2016 at 6:31 PM, Remi Galan Alfonso
> <remi.galan-alfonso@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> > Elena Petrashen <elena.petrashen@xxxxxxxxx> wrote:
> >> +void delete_branch_advice(const char *name, const char *ref)
> >> +{
> >> +        const char fmt[] =
> >> +        "\nNote: to restore the deleted branch:\n\ngit branch %s %s\n";
> >
> > Shouldn't that be marked for translation, like is done with the other
> > strings?
> >
> > Thanks,
> > Rémi
> 
> Thank you for letting me know about that! Could you please
> help me out and explain how do I mark it for translation? I tried
> to do it the same way as with the other strings but evidently
> didn't quite succeed.

I am not sure.
I tried to grep similar cases, it seems that you can do the following:

	const char fmt[] = N_("\nNote: to restore [...] \ngit branch %s %s\n");
	fprintf(stderr, _(fmt), name, ref);

Some similar example in builtin/add.c:

	static const char ignore_error[] =
	N_("The following paths are ignored by one of your .gitignore files:\n");
	[...]
			fprintf(stderr, _(ignore_error));

Or you can define fmt as a 'const char *' and in that case do the
following:

	const char *fmt = _("\nNote: to restore [...] \n git branch %s %s\n");
	fprintf(stderr, fmt, name, ref);


In builtin/am.c:
		const char *invalid_line = _("Malformed input line: '%s'.");
	[...]
				ret = error(invalid_line, sb.buf);

I don't know which one is the best way to go though.

Thanks,
Rémi
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]