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.



Reply via email to