On 2014-12-21 14:22:10 -0500, Andrew Dunstan wrote:
> @@ -301,11 +301,11 @@ slashUsage(unsigned short int pager)
>   * show list of available variables (options) from command line
>   */
>  void
> -helpVariables(unsigned short int pager)
> +helpVariables(unsigned short int pager,      int pager_min_lines)
>  {
>       FILE       *output;

Odd space before pager_min_lines.

Without having actually tried it, it looks clean enough to me. If
there's more pager options we might at some point introduce a pager
options struct instead adding more options to PageOutput. But for now it
seems ok enough.

Greetings,

Andres Freund


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to