Re: merge psql ef/ev sf/sv handling functions
Fabien COELHO <coelho@cri.ensmp.fr>
From: Fabien COELHO <coelho@cri.ensmp.fr>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: PostgreSQL Developers <pgsql-hackers@postgresql.org>
Date: 2017-09-07T05:19:59Z
Lists: pgsql-hackers
> you can't do this sort of thing:
>
> - psql_error("The server (version %s) does not support editing function source.\n",
> + psql_error("The server (version %s) does not support editing %s.\n",
> formatPGVersionNumber(pset.sversion, false,
> - sverbuf, sizeof(sverbuf)));
> + sverbuf, sizeof(sverbuf)),
> + is_func ? "function source" : "view definitions");
>
> It's too much of a pain in the rear for translators.
Argh, indeed, I totally forgot about translations. Usually there is a _()
hint for gettext.
> See
> https://www.postgresql.org/docs/devel/static/nls-programmer.html#nls-guidelines
> Usually we just use two independent messages, and that's what I did.
Yep, makes sense. Thanks for the fix.
--
Fabien.
Commits
-
Merge duplicative code for \sf/\sv, \ef/\ev in psql/command.c.
- ca4e20fde87d 11.0 landed