Don't elide blank lines when accumulating psql command history.

robertmhaas · robertmhaas · commit d2202a698736 · 2011-11-15T20:36:32.000-05:00
This can change the meaning of queries, if the blank line happens to
occur in the middle of a quoted literal, as per complaint from Tomas Vondra.

Back-patch to all supported branches.
diff --git a/src/bin/psql/input.c b/src/bin/psql/input.c
@@ -86,10 +86,10 @@ void
 pg_append_history(const char *s, PQExpBuffer history_buf)
 {
 #ifdef USE_READLINE
-	if (useHistory && s && s[0])
+	if (useHistory && s)
 	{
 		appendPQExpBufferStr(history_buf, s);
-		if (s[strlen(s) - 1] != '\n')
+		if (!s[0] || s[strlen(s) - 1] != '\n')
 			appendPQExpBufferChar(history_buf, '\n');
 	}
 #endif

Original file line number	Diff line number	Diff line change
`@@ -86,10 +86,10 @@ void`
`86`	`86`	`pg_append_history(const char *s, PQExpBuffer history_buf)`
`87`	`87`	`{`
`88`	`88`	`#ifdef USE_READLINE`
`89`		`- if (useHistory && s && s[0])`
	`89`	`+ if (useHistory && s)`
`90`	`90`	`{`
`91`	`91`	`appendPQExpBufferStr(history_buf, s);`
`92`		`- if (s[strlen(s) - 1] != '\n')`
	`92`	`+ if (!s[0] \|\| s[strlen(s) - 1] != '\n')`
`93`	`93`	`appendPQExpBufferChar(history_buf, '\n');`
`94`	`94`	`}`
`95`	`95`	`#endif`