Tab completion of double quoted identifiers broken - Mailing list pgsql-hackers

From Dean Rasheed
Subject Tab completion of double quoted identifiers broken
Date
Msg-id CAEZATCXutha5_fD5Pz+Ken9yiaCwt6o+uCUAupEwsx-Mb_1oyQ@mail.gmail.com
Whole thread Raw
Responses Re: Tab completion of double quoted identifiers broken  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Hi,

I just spotted that tab completion of double quoted identifiers seems
to be broken in 9.2devel. For example things like this which worked in
9.1 no longer work:

UPDATE "foo bar" <tab>

It looks like the problem is in get_previous_words() here:
               if (buf[start] == '"')                   inquotes = !inquotes;               else if (!inquotes)
     {                   ... test for start of word ...
 

which fails to account for the fact that the double quote itself might
be the start of the word. I think the solution is just to remove the
"else":
               if (buf[start] == '"')                   inquotes = !inquotes;               if (!inquotes)
{                   ... test for start of word ...
 

to allow it to find a word-breaking character immediately before the
double quote.

Regards,
Dean


pgsql-hackers by date:

Previous
From: Hitoshi Harada
Date:
Subject: Re: Odd out of memory problem.
Next
From: Simon Riggs
Date:
Subject: Re: measuring lwlock-related latency spikes