Re: psql tab completion bug and possible fix

From: Ian Barwick <barwick(at)gmx(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: psql tab completion bug and possible fix
Date: 2003-10-15 21:04:35
Message-ID: 200310152304.35580.barwick@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Wednesday 15 October 2003 22:45, Tom Lane wrote:
> Ian Barwick <barwick(at)gmx(dot)net> writes:
> > On Tuesday 14 October 2003 23:38, Tom Lane wrote:
> >> Surely all those replacements of \\ with \\\\ are wrong.
> >
> > The slash in the slash command gets escaped too...:
>
> Not the way I did it. You were doing the escaping in the wrong place
> IMHO --- the string passed to _complete_from_query() mustn't be escaped
> already, because it is not only used to send a command to the backend,
> but also to compare against the strings returned by the backend, which
> won't be escaped. So the escaping needs to be done internally to
> _complete_from_query(), and that eliminates the side-effects elsewhere.

Aha, sorry, I hadn't seen your patch. It works excellently for me, so
I shall gripe no longer ;-).

Many thanks

Ian Barwick
barwick(at)gmx(dot)net

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2003-10-16 03:47:10 Re: release.sgml, minor pg_autovacuum changes
Previous Message Tom Lane 2003-10-15 20:45:19 Re: psql tab completion bug and possible fix