From: | Corey Huinker <corey(dot)huinker(at)gmail(dot)com> |
---|---|
To: | Erik Rijkers <er(at)xs4all(dot)nl> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Daniel Verite <daniel(at)manitou-mail(dot)org>, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, PostgreSQL <pgsql-hackers(at)postgresql(dot)org>, pgsql-hackers-owner(at)postgresql(dot)org |
Subject: | Re: \if, \elseif, \else, \endif (was Re: PSQL commands: \quit_if, \quit_unless) |
Date: | 2017-02-10 05:13:30 |
Message-ID: | CADkLM=fbsnkWUWOKz8JiU5hN-8mKDm7MgsuAq6pHk-22eqJ3xA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Feb 9, 2017 at 4:43 PM, Erik Rijkers <er(at)xs4all(dot)nl> wrote:
> On 2017-02-09 22:15, Tom Lane wrote:
>
>> Corey Huinker <corey(dot)huinker(at)gmail(dot)com> writes:
>>
>
> The feature now ( at patch v10) lets you break off with Ctrl-C anywhere.
> I like it now much more.
>
> The main thing I still dislike somewhat about the patch is the verbose
> output. To be honest I would prefer to just remove /all/ the interactive
> output.
>
> I would vote to just make it remain silent if there is no error. (and if
> there is an error, issue a message and exit)
>
> thanks,
>
> Erik Rijkers
>
Changes in this patch:
- invalid boolean expression on \if or \elif is treated as if the script
had a bad \command, so it either stops the script (ON_ERROR_STOP, script
mode), or just gives the ParseVariableBool error and continues.
- All interactive "barks" removed except for
"command ignored. use \endif or Ctrl-C to exit current branch" when the
user types a non-branching \command in a false branch
"query ignored. use \endif or Ctrl-C to exit current branch" when the
user types a non-branching \command in a false branch
"\if: escaped" when a user does press Ctrl-C and they escape a branch.
- remaining error messages are tersed:
\elif: cannot occur after \else
\elif: no matching \if
\else: cannot occur after \else
\else: no matching \if
\endif: no matching \if
found EOF before closing \endif(s)
Attachment | Content-Type | Size |
---|---|---|
0001.if_endif.v11.diff | text/plain | 25.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2017-02-10 05:32:14 | Re: contrib modules and relkind check |
Previous Message | Michael Paquier | 2017-02-10 05:00:16 | Re: possibility to specify template database for pg_regress |