On Fri, Feb 2, 2018 at 12:39:28AM -0500, Bruce Momjian wrote:
> I just thought of an inconsistency. First, we now consistently exit with
> 'exit', 'quit', and '\q' if used in an empty psql query buffer. Also, we now
> hint when 'exit' and 'quit' are used in non-empty query buffers:
>
> test=> SELECT
> test-> exit
> --> Use \q to quit.
>
> We obviously don't need to hint for '\q' since it works in this context.
>
> Where it is odd is that we hint for 'exit' and 'quit' in places that
> '\q' doesn't work, but we don't hint for '\q' in the same contexts.
> Here is the hint for 'exit' and 'quit':
>
> test=> SELECT '
> test'> exit
> --> Use control-D to quit.
>
> but we don't hint for '\q':
>
> test=> SELECT '
> test'> \q
> --> test'>
>
> Should we give the same "Use control-D to quit." hint here for '\q'?
> I think it is logical that we should.
I have applied the attached patch giving a ^D/^C hint where \q is
ignored, and add C comment documenting why exit/quit is not documented.
I consider this issue closed, or quited, or exited. :-)
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +