Re: BUG #16059: Tab-completion of filenames in COPY commands removesrequired quotes - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: BUG #16059: Tab-completion of filenames in COPY commands removesrequired quotes
Date
Msg-id d746fc4c-c47d-2581-e9b2-641b05e350e5@2ndquadrant.com
Whole thread Raw
In response to Re: BUG #16059: Tab-completion of filenames in COPY commands removes required quotes  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: BUG #16059: Tab-completion of filenames in COPY commands removes required quotes
List pgsql-hackers
On 2020-01-14 02:38, Tom Lane wrote:
> I wrote:
>> Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
>>> I have found a weird behavior with identifier quoting, which is not the
>>> subject of this patch, but it appears to be affected by it.
> 
>> I'll think about how to improve this.  Given that we're dequoting
>> filenames explicitly anyway, maybe we don't need to tell readline that
>> double-quote is a quoting character.  Another idea is that maybe
>> we ought to refactor things so that identifier dequoting and requoting
>> is handled explicitly, similarly to the way filenames work now.
>> That would make the patch a lot bigger though.
> 
> On reflection, it seems like the best bet for the moment is to
> remove double-quote from the rl_completer_quote_characters string,
> which should restore all behavior around double-quoted strings to
> what it was before.  (We have to keep single-quote in that string,
> though, or quoted file names fail entirely.)

This patch (version 6) looks good to me.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: Re: table partitioning and access privileges
Next
From: Christoph Berg
Date:
Subject: Re: libxml2 is dropping xml2-config