On 03.06.22 22:51, Tom Lane wrote:
> + HELP0(" -c, --command=COMMAND run only single command (SQL or internal) and exit\n");
> + HELP(" -d, --dbname=DBNAME database name to connect to (default: \"%s\")\n",
> + env);
I wonder whether this mix of HELP0 and HELP is necessary. The original
code didn't care about calling fprintf even if there are no
substitutions. I think this could lead to misalignment errors. I
vaguely recall we once had mixes of fprintf and fputs and got rid of
them for this reason.