Re: review: psql and pset without any arguments

Le 30/09/2013 05:43, Alvaro Herrera a écrit :
> Gilles Darold escribió:
>
>> + else if (strcmp(param, "numericlocale") == 0)
>> + {
>> + if (popt->topt.numericLocale)
>> + puts(_("Locale-adjusted numeric output (numericlocale) is on."));
>> + else
>> + puts(_("Locale-adjusted numeric output (numericlocale) is off."));
>> + }
> Please don't make the variable name part of the translatable message. I
> suggest using the following pattern:
>
>> + else if (strcmp(param, "numericlocale") == 0)
>> + {
>> + if (popt->topt.numericLocale)
>> + printf(_("Locale-adjusted numeric output (%s) is on."), "numericlocale");
>> + else
>> + printf(_("Locale-adjusted numeric output (%s) is off."), "numericlocale");
>> + }
> Otherwise it will be too easy for the translator to make the mistake
> that the variable name needs translation too.
>

That's right, here is the patch modified with just a little change with
your suggestion:

if (popt->topt.numericLocale)
printf(_("Locale-adjusted numeric output (%s) is
on.\n"), param);
else
printf(_("Locale-adjusted numeric output (%s) is
off.\n"), param);

Thanks

--
Gilles Darold
Administrateur de bases de données
http://dalibo.com - http://dalibo.org