Re: [Fwd: Re: proposal: new long psql parameter --on-error-stop]

OK, let me help you, though I'm only a Japanese who is never confident in my
English.

(1)
As Fujii-san pointed out, could you add explanation for --help-variables in
doc/src/sgml/ref/psqlref.sgml?

(2)
+ printf(_(" --help-variables list of available configuration
variables (options), then exit\n"));

should better be:

+ printf(_(" --help-variables show A list of all specially treated
variables, then exit\n"));

This follows the psql manual page. Similarly,

+ printf(_("List of variables (options) for use from command line.\n"));

should be:

+ printf(_("List of specially treated variables.\n"));

(3)
+ printf(_(" ECHO control what input can be writtent to
standard output [all, queries]\n"));

"writtent" should be "written". "controls" should be "control" like other
options.

(4)
+ printf(_(" ECHO_HIDDEN display internal queries (same as -E
option)\n"));

should better be:

+ printf(_(" ECHO_HIDDEN display internal queries executed by
backslash commands\n"));

I think "(same as ...)" can be omitted to keep the description short. If
you want to retain it, other variables should also accompany similar
description, such as -a for ECHO.

(5)
+ printf(_(" FETCH_COUNT fetch many rows at a time (use less memory)
(default 0 unlimited)\n"));

should better be:

+ printf(_(" FETCH_COUNT the number of result rows to fetch and
display at a time (default: 0=unlimited)\n"));

(6)
+ printf(_(" HISTCONTROL when set, controls history list
[ignorespace, ignoredups, ignoreboth]\n"));

should better be:

+ printf(_(" HISTCONTROL control history list [ignorespace,
ignoredups, ignoreboth]\n"));

(7)
+ printf(_(" USER the database user currently connected\n"));

should add "as" at the end:

+ printf(_(" USER the database user currently connected
as\n"));

(8)
"Printing options" section lack the following ones described in psql manual:

columns
expanded (or x)
footer
numericlocale
tableattr (or T)

(9)
+ printf(_("\nEnvironment options:\n"));

should be ""Environment variables". And this section lacks description for
Windows, such as:

+ printf(_(" NAME=VALUE [NAME=VALUE] psql ...\n or \\setenv NAME [VALUE]
in interactive mode\n\n"));
+ printf(_(" PGPASSFILE password file (default ~/.pgpass)\n"));

Regards
MauMau