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.