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.
